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

[AO] Improve loading state of observability overview page #150305

Closed
maryam-saeidi opened this issue Feb 6, 2023 · 1 comment · Fixed by #150327
Closed

[AO] Improve loading state of observability overview page #150305

maryam-saeidi opened this issue Feb 6, 2023 · 1 comment · Fixed by #150327
Assignees
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

Currently, we have a different way of showing the loading state on the overview page for different sections. In this ticket, we need to check how this loading happens in a different section and how to use the same pattern everywhere.

We need to also update the Loading keyword to show something similar to other sections:

image

✅ Acceptance Criteria

  • Unify the loading state on the observability overview page
@maryam-saeidi maryam-saeidi added Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0 labels Feb 6, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@maryam-saeidi maryam-saeidi self-assigned this Feb 6, 2023
maryam-saeidi added a commit that referenced this issue Feb 8, 2023
Fixes #150305 

## Summary

This PR fixes the loading state of the `infra_metrics` section.

I also tried to check why the `Log` section appears without any loading
state but it seems the API that returns if this section has data takes a
bit more time than the other APIs (@weltenwort please let me know if my
assumption is not correct or if you have suggestion about how to get the
hasData info for `Log` section quicker)

|Before|After|
|---|---|

|![image](https://user-images.githubusercontent.com/12370520/216984455-f9277a88-0759-4e5e-b8aa-6b8af745a82f.png)|![image](https://user-images.githubusercontent.com/12370520/216984578-a7392b90-3709-4487-9b9f-8bf70ddaffe5.png)|
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants