-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added new XSIAM content items - case fields, layouts and layout rules #4240
Conversation
Changelog(s) in markdown:
|
Changelog(s) in markdown:
|
self.data["group"] != "incident" | ||
and self.data["group"] != "indicator" | ||
and self.data["group"] != "case" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pairing tends to repeat itself (incident + indicator + case) - usually when there's such a strong pairing it's better to unify them under a common constant, and use the constant for conditions.
Let's discuss this offline
Changelog(s) in markdown:
|
…nto CIAC-9930-xsiam_content_items
Changelog(s) in markdown:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done!
Related Issues
fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-9930
Description
Added support for new Case content items: Fields, Layouts and Layout rules.
bucket link: https://console.cloud.google.com/storage/browser/marketplace-ci-build/content/builds/CIAC-9930-xsiam_content_items/1062944/marketplacev2;tab=objects?prefix=&forceOnObjectsSortingFiltering=false