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.

Sign up for GitHub

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

Jump to bottom

Odyssey Stats: Add loading spinner or similar for Stats pages and/or components #70509

Closed
kangzj opened this issue Nov 29, 2022 · 2 comments
Closed

Odyssey Stats: Add loading spinner or similar for Stats pages and/or components #70509

kangzj opened this issue Nov 29, 2022 · 2 comments
Labels
[Size] M Medium sized issue Stats Everything related to our analytics product at /stats/ [Type] Enhancement

Comments

@kangzj
Copy link
Contributor

kangzj commented Nov 29, 2022 •
edited
Loading

The requests from Odyssey Stats need to be proxied to WPCOM, which greatly increase the latency of the response of the pages. The components are blank while loading data.

Calypso Stats seems okay tho benefiting from accessing public-api directly.
@kangzj kangzj added Stats Everything related to our analytics product at /stats/ [Size] M Medium sized issue [Type] Enhancement labels Nov 29, 2022
@kangzj kangzj changed the title Add loading spinner for Stats pages and/or components Add loading spinner or similar for Stats pages and/or components Nov 29, 2022
@jsnmoon jsnmoon changed the title Add loading spinner or similar for Stats pages and/or components Odyssey Stats: Add loading spinner or similar for Stats pages and/or components Jan 11, 2023
@wilmersondasilva
Copy link
Contributor

wilmersondasilva commented Jan 13, 2023 •
edited
Loading

@kangzj I'm not sure if I got this issue. When I access the Odyssey Stats, I see a loading spinner. Can you provide more details or attach a video?

Image

@kangzj
Copy link
Contributor Author

kangzj commented Jan 13, 2023

@wilmersondasilva thanks for looking into this. This was addressed in Automattic/jetpack#28219 and I’m closing it now.

@kangzj kangzj closed this as completed Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[Size] M Medium sized issue Stats Everything related to our analytics product at /stats/ [Type] Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kangzj @wilmersondasilva