Skip to content

[8.x] [ResponseOps][Rules] Add loading state to rule params data views selector (#203654)#204424

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-203654
Dec 16, 2024
Merged

[8.x] [ResponseOps][Rules] Add loading state to rule params data views selector (#203654)#204424
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-203654

Conversation

@kibanamachine
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

…ctor (elastic#203654)

## Summary

Introduces a loading state in the data views select popover and renders
a loading indicator when DVs are not available yet. This makes sure that
even if the `savedObjectsClient.find` call of the data views service
takes a long time, we don't show an empty popover in the meantime.

https://github.com/user-attachments/assets/5bbe0c68-3ceb-4d7f-91fd-357db4caa5c1

## References

Fixes elastic#198502

## Release note

Fix race condition in alerting rules data view selector

(cherry picked from commit 713d4bb)
@kibanamachine kibanamachine merged commit ab57078 into elastic:8.x Dec 16, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
stackAlerts 25.1KB 25.2KB +103.0B

cc @umbopepato

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