Skip to content

[8.19] [Observability Alerting] Refetch alert detail rule data on edit flyout submit (#222118)#225131

Merged
justinkambic merged 1 commit intoelastic:8.19from
justinkambic:backport/8.19/pr-222118
Jun 24, 2025
Merged

[8.19] [Observability Alerting] Refetch alert detail rule data on edit flyout submit (#222118)#225131
justinkambic merged 1 commit intoelastic:8.19from
justinkambic:backport/8.19/pr-222118

Conversation

@justinkambic
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…t submit (elastic#222118)

## Summary

Resolves elastic#221428.

We noticed recently when we added the Investigation Guide that, if the
user edits the values in the guide using the edit flyout, the data on
the main page will not refresh when they submit.

This patch would add an `onUpdate` handler prop to the Header Actions
component that would enable it to trigger the refresh on the parent
component when the user submits the form in the edit flyout.

Example:

![20250530164523](https://github.com/user-attachments/assets/bd796824-e0d6-464d-b10e-89537d9b590e)

(cherry picked from commit 1886697)
@justinkambic justinkambic added the backport This PR is a backport of another PR label Jun 24, 2025
@justinkambic justinkambic enabled auto-merge (squash) June 24, 2025 17:34
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Jun 24, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

Copy link
Member

@jasonrhodes jasonrhodes left a comment

Choose a reason for hiding this comment

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

LGTM - did not run locally but approving a backport after discussing with Justin

@justinkambic justinkambic merged commit ed586f5 into elastic:8.19 Jun 24, 2025
13 checks passed
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 Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants