Skip to content

[8.x] [Security Solution][Alert Flyout] Update entity insight badge to open entity flyouts (#208287)#210476

Merged
christineweng merged 1 commit intoelastic:8.xfrom
christineweng:backport/8.x/pr-208287
Feb 11, 2025
Merged

[8.x] [Security Solution][Alert Flyout] Update entity insight badge to open entity flyouts (#208287)#210476
christineweng merged 1 commit intoelastic:8.xfrom
christineweng:backport/8.x/pr-208287

Conversation

@christineweng
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

… entity flyouts (elastic#208287)

## Summary

This PR updates the cloud insights in entity section to open to entity
flyout. When examining the insights (for example, the host is shown to
have 50 alerts, instead of opening the preview, and user click on the
details and go to respective tabs, this PR updated the behavior to open
the details tab via 1 click. The goal is to reduce friction for users
during investigation.

Feature flag: `newExpandableFlyoutNavigationEnabled`

When flag is off:
- Click on alert count should open timeline (if user has timeline
privileage)
- Click on misconfigurations and vulnerabilities badge should open
host/user preview

https://github.com/user-attachments/assets/23e0cc40-129d-4e75-b5be-26a49dcad710

When flag is on:
- Click on count badges should open the respective entity flyout and the
insights details tab

https://github.com/user-attachments/assets/5dfc39b7-edae-4b76-9a3a-79326337cb3b

### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 6e61f52)

# Conflicts:
#	x-pack/solutions/security/plugins/security_solution/public/flyout/document_details/shared/components/alert_count_insight.tsx
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #42 / alerting api integration security and spaces enabled - Group 2 Alerting and Actions Telemetry test telemetry should retrieve telemetry data in the expected format
  • [job] [logs] FTR Configs #25 / dashboard app - esql controls dashboard - add an value type ES|QL control should update the Lens chart accordingly

Metrics [docs]

Async chunks

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

id before after diff
securitySolution 18.7MB 18.7MB +16.1KB

@christineweng christineweng merged commit f5383e7 into elastic:8.x Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants