-
Notifications
You must be signed in to change notification settings - Fork 191
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] Entity dashboard #2565
[DOCS] Entity dashboard #2565
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.
Looks great, left some minor suggestions for your consideration. Some may be a bit nitpicky so please feel free to "resolve". Nice use of the requirements block, and I like the tip at the bottom.
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.
Added a few comments, 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.
Just some light edits, looks great overall!
Hi @jmikell821 , We have validated above preview link and found an issue. We will re-validate this PR once above issue is fixed. Thanks ! |
Hi @karanverma-qasource -- yes, I added a comment to myself here as a reminder. Once #2580 is merged, I'll add the anchor ref link to the topic here. |
* Adding content for the entity dashboard. * Merging round 1 of feedback. * Update entity-dashboard.asciidoc * Adding host risk score reference. (cherry picked from commit e14b18d)
* Adding content for the entity dashboard. * Merging round 1 of feedback. * Update entity-dashboard.asciidoc * Adding host risk score reference. (cherry picked from commit e14b18d) Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Resolves #2517.
Preview here.