-
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
Update alerting concepts diagram #86699
Update alerting concepts diagram #86699
Conversation
Pinging @elastic/kibana-docs (Team:Docs) |
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
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! Updated description is much clearer!
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! Thank you for doing this!
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.
oooooh that's how it works! 😆
Thanks Peter, this is way better.
I update the summary section of the alerting getting started guide. The summary section did OK at summarizing the concepts, but it didn't do a good job of explaining how they were related.
I've updated the diagram with a numbered sequence showing the steps, and each of the 4 steps is also described in text below the diagram. I feel this does a better job of showing how the concepts are related.
Current:

Proposed:
