-
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] Alert counts added to Explore pages #2576
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.
Solid updates, looks good! Added a few line-edits, etc.
Also: almost all the section headings here are just bold lines of text, not actual Asciidoc syntax for section headings & anchors. It looks like they've been this way for a long time, so we should probably either fix them now in this PR, or create a docs issue to make sure we get them fixed after the release crunch.
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.
Just left a few minor comments. Overall, LGTM!
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.
Looks good! I left some suggestions for your consideration, hope they are helpful.
Hi @jmikell821 we have validated the shared doc preview links and doc information is complete and good to go. Build Details:
Screen-Shot Hence we are closing this issue and adding "QA:Validated" tag to it. thanks !! |
* Publishing edits. * Slight edits. * Merging feedback. * Merging addt'l feedback. (cherry picked from commit f0cc44b)
Resolves #2525.
Previews: