-
Notifications
You must be signed in to change notification settings - Fork 8k
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] Alert Grouping - fix loading issues #152235
[Security solution] Alert Grouping - fix loading issues #152235
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
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!
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit 3d649ef) # Conflicts: # x-pack/plugins/security_solution/public/detections/components/alerts_table/grouped_alerts.tsx # x-pack/plugins/security_solution/public/detections/components/alerts_table/index.test.tsx
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
… (#152349) # Backport This will backport the following commits from `main` to `8.7`: - [[Security solution] Alert Grouping - fix loading issues (#152235)](#152235) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Steph Milovic","email":"stephanie.milovic@elastic.co"},"sourceCommit":{"committedDate":"2023-02-28T13:16:39Z","message":"[Security solution] Alert Grouping - fix loading issues (#152235)","sha":"3d649ef254ae93a06bea2916686fca3a519b89d0","branchLabelMapping":{"^v8.8.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Threat Hunting","Team: SecuritySolution","Team:Threat Hunting:Explore","v8.7.0","v8.8.0"],"number":152235,"url":"#152235 solution] Alert Grouping - fix loading issues (#152235)","sha":"3d649ef254ae93a06bea2916686fca3a519b89d0"}},"sourceBranch":"main","suggestedTargetBranches":["8.7"],"targetPullRequestStates":[{"branch":"8.7","label":"v8.7.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.8.0","labelRegex":"^v8.8.0$","isSourceBranch":true,"state":"MERGED","url":"#152235 solution] Alert Grouping - fix loading issues (#152235)","sha":"3d649ef254ae93a06bea2916686fca3a519b89d0"}}]}] BACKPORT--> Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Fixes #150726 by adding the
isLoading
property to the accordion. I also added anEuiProgress
loader when there are not yet results but we are loadingFixes #150832 by removing code in
grouped_alerts
that returned null forloading
. This caused the groups to rerender and the accordion to close.Grouping loading state
loading2.mov
Building block toggle loading state
Building block highlighting is broken for another reason #152318
building.mov