-
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
Soc dashboard - UI #128087
Soc dashboard - UI #128087
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
/oblt-deploy |
@@ -17,7 +17,7 @@ export const allowedExperimentalValues = Object.freeze({ | |||
tGridEventRenderedViewEnabled: true, | |||
excludePoliciesInFilterEnabled: false, | |||
usersEnabled: true, | |||
detectionResponseEnabled: false, | |||
detectionResponseEnabled: true, |
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.
should this merge to main as true? Should the feature still be hidden or am I misunderstanding what this is for?
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.
Nah, it should be false, enabled it for deploying to cloud instance for design review, will remember to disable that
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.
good to merge :D
x-pack/plugins/security_solution/common/experimental_features.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/common/components/charts/donutchart.tsx
Outdated
Show resolved
Hide resolved
…ts/donutchart.tsx Co-authored-by: Sergi Massaneda <sergi.massaneda@gmail.com>
@elasticmachine merge upstream |
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 @angorayc, great work! the donut chart has been hard to tame.
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.
Given feedback on spacing, alignment and font sizes.
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
* init pie chart * styling * donut card * styling * parser * wording * unit tests * disable detectionResponse feature flag * rename props * add detection_response folder * add query hook * disable detectionResponse * alerts button and count * toggle query * fix unit tests * disable dashboard * fillColor * disable dashboard * fix legends * detectionResponseEnabled * review * rm old tests * review * fix types * legend * rename * rm i18n keys * unit tests * update i18n keys * add unit tests * fix unit tests * review * review * styling * [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix' * styling * styling * useEuiTheme * useEuiTheme * enable feature flag * disable flag * enable dashboard * disable dashboard * Update x-pack/plugins/security_solution/public/common/components/charts/donutchart.tsx Co-authored-by: Sergi Massaneda <sergi.massaneda@gmail.com> * styling * styling * unit tests * unit tests * unit test * unit tests Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Sergi Massaneda <sergi.massaneda@gmail.com>
Summary
#127649
This UI is to render donut charts with alerts by status (mock) data.
It shows an empty donut if a bucket is missing:
Kapture.2022-04-08.at.02.36.18.mp4
Delete any items that are not applicable to this PR.