[AO] Improve loading state of metric threshold preview chart #152438
Assignees
Labels
chore
Team: Actionable Observability - DEPRECATED
For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge"
v8.8.0
Comments
maryam-saeidi
added
chore
Team: Actionable Observability - DEPRECATED
|
Pinging @elastic/actionable-observability (Team: Actionable Observability) |
This was referenced Mar 6, 2023
maryam-saeidi
added a commit
that referenced
this issue
Mar 22, 2023
Resolves #152438 ## 📝 Summary In this PR, I used react query to improve the loading of preview and metric explorer charts. Also, I used the `EuiLoadingChart` instead of showing a text. ### Before https://user-images.githubusercontent.com/12370520/224107429-16bd6f1c-a1b7-429c-b56e-e2201f7d767c.mov ### After https://user-images.githubusercontent.com/12370520/224107649-c8b5c9a3-d371-405d-a0eb-8c1151047437.mov ## 🧪 How to test - Open a metric threshold/ inventory rule creation flyout and check the preview charts - They should work as before - Check the metric explorer page charts - Adding a filter, changing the time range, or adding a group should work as before https://user-images.githubusercontent.com/12370520/224105438-0ffbd1e4-464f-4438-9894-939f5768c4f4.mov
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
📝 Summary
At the moment, while loading the metric threshold preview chart, first we see
no date, thenLoadingkeyword, and then the chart as shown below:Screen.Recording.2023-03-01.at.11.58.08.mov
In this ticket, we want to improve this by using a loader and not showing no data at the start.
✅ Acceptance Criteria
The text was updated successfully, but these errors were encountered: