-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Show friendly error message on API fetch failure on stats #17570
Show friendly error message on API fetch failure on stats #17570
Conversation
Thank you for opening this PR! We appreciate you! For all pull requests coming from third-party forks we will need to A Forem Team member will review this contribution and get back to |
040ebfa
to
91e987f
Compare
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
91e987f
to
edb09f6
Compare
This PR is now ready for review. |
@forem-team Thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the walkthrough of the code. This looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this and it works great, thank you @tnir ✨ I'll merge it later today 😊
Thank you for merging, @Ridhwana! |
Signed-off-by: Takuya Noguchi takninnovationresearch@gmail.com
What type of PR is this? (check all applicable)
Description
Shows friendly error message on API fetch failures on
/:user/:slug/stats
.Related Tickets & Documents
QA Instructions, Screenshots, Recordings
/orville_rippin/a-summer-bird-cage-quibusdam-impedit-47p9/stats
on your dev server.cf. Another screenshot (with 1024px screen width):
Although I know the team is getting rid of Bootstrap CSS (#17412), Bootstrap is used additionally in this PR as it is already used on this page.
UI accessibility concerns?
n/a
Added/updated tests?