-
Notifications
You must be signed in to change notification settings - Fork 195
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
[DOCS] Bulk edit rules updates (rule schedules, actions) #2567
Conversation
Documentation previews: |
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.
One trivial suggestion for your consideration, otherwise LGTM
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
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.
LGTM
Hi Team, We have validated above PR and the changes are good to go. Hence, marking it as 'QA Validated. Thanks ! |
* Add rule actions option * Add rule schedules option * Add note on action frequency * Lil edit * Update docs/detections/rules-ui-manage.asciidoc Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com> * Revise note Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com> (cherry picked from commit 659de03)
* Add rule actions option * Add rule schedules option * Add note on action frequency * Lil edit * Update docs/detections/rules-ui-manage.asciidoc Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com> * Revise note Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com> (cherry picked from commit 659de03) Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
Related issues — Streamlined into a single PR:
Preview: Manage detection rules | Modify existing rules settings