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

[ResponseOps][Alerts] Add fallback value for alert filter controls panels #193745

Conversation

umbopepato
Copy link
Member

Summary

Adds a fallback value for the alert filter controls bar's panels configuration when it's undefined.

Fixes #193565

@umbopepato umbopepato added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.15.0 labels Sep 23, 2024
@umbopepato umbopepato requested a review from a team as a code owner September 23, 2024 15:46
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@umbopepato umbopepato added release_note:fix backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development labels Sep 23, 2024
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #39 / discover/esql discover esql view ES|QL in Discover should render esql view correctly

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 20.4MB 20.4MB +34.0B
triggersActionsUi 1.6MB 1.6MB +34.0B
total +68.0B

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

@@ -17,7 +17,7 @@ import { isEmpty, isEqual, pick } from 'lodash';
import type { FilterControlConfig } from './types';

export const getPanelsInOrderFromControlsState = (controlState: ControlGroupRuntimeState) => {
const panels = controlState.initialChildControlState;
const panels = controlState.initialChildControlState ?? {};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to have some tests for the util.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@umbopepato
Copy link
Member Author

Closing in favor of #194785

@umbopepato umbopepato closed this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development release_note:fix Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.15.0 v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ResponseOps][Alerts] Stack alerts page crashes on certain local storage configurations
5 participants