-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Improves Detections API docs content #187224
[DOCS] Improves Detections API docs content #187224
Conversation
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
Hi @tomsonpl - I replaced myself with you on this PR because I think you are the most knowledgable on these changes. Looks like there is only 4 files we are code owners for. There was a changes to the rule automated response actions schema (for OSQuery I think). Hope you don't mind 🙂 |
...y_solution/common/api/detection_engine/alert_assignees/set_alert_assignees_route.schema.yaml
Outdated
Show resolved
Hide resolved
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.
Osquery related changes LGTM, thanks!
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, thanks!
...y_solution/common/api/detection_engine/alert_assignees/set_alert_assignees_route.schema.yaml
Outdated
Show resolved
Hide resolved
...ty_solution/common/api/detection_engine/alert_tags/set_alert_tags/set_alert_tags.schema.yaml
Outdated
Show resolved
Hide resolved
⏳ Build in-progress, with failures
Failed CI StepsHistory |
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.
@natasha-moore-elastic thank you for this PR 🙏
I like reworked summaries and descriptions. I didn't know that description block like below
> info
> You cannot add and remove the same assignee in the same request.
transforms into an info section in Bump.sh
Summary
Resolves https://github.com/elastic/docs-projects/issues/219 by improving the Detections API docs content. Adds missing and improves existing operation summaries and operation descriptions.
Also addresses some of the work planned in #183702.