Skip to content
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

[Security Solution] expandable flyout - add investigate in timeline for alert count and document count prevalence details table columns #165054

Conversation

PhilippeOberti
Copy link
Contributor

Summary

This PR adds the missing interaction within the prevalence table in the expandable flyout left section.
The users can now click on the alert count and document count columns to view the results in timeline:

  • alert count clicks open timeline with the highlighted field key/value pair
  • document count clicks open timeline with the highlighted field key/value pair as well as event.kind !== signal
Screen.Recording.2023-08-29.at.12.52.04.AM.mov

Fixes #164974

Checklist

Delete any items that are not applicable to this PR.

@PhilippeOberti PhilippeOberti requested a review from a team as a code owner August 28, 2023 22:55
@PhilippeOberti PhilippeOberti force-pushed the expandable-flyout-prevalence-investigate-in-timeline branch from 128f7ec to 9eb5367 Compare August 28, 2023 22:59
@PhilippeOberti PhilippeOberti added release_note:feature Makes this part of the condensed release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.10.0 release_note:enhancement and removed release_note:feature Makes this part of the condensed release notes labels Aug 28, 2023
@kibana-ci
Copy link
Collaborator

kibana-ci commented Aug 28, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #18 / PrevalenceDetails should render the table
  • [job] [logs] Serverless Observability Tests / serverless common UI Management Disabled UIs Rollup Jobs is not accessible

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 4477 4476 -1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 12.6MB 12.6MB +188.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@PhilippeOberti PhilippeOberti force-pushed the expandable-flyout-prevalence-investigate-in-timeline branch from 9eb5367 to 158f981 Compare August 29, 2023 07:08
@PhilippeOberti
Copy link
Contributor Author

@christineweng thanks for working out the code. I'm closing this PR as it is a duplicate of this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants