-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Blank Achievements activity for some users #2042
Comments
* Added a sticky snack bar in achievements activity which shows up when fetch api fails * Snackbar comes with an action button which on click retries fetch. This way user is never shown a blank screen and always has an option to retry fetches
I checked by calling the API for this user and it seems there's a lot of data corresponding to this user. https://tools.wmflabs.org/urbanecmbot/commonsmisc/feedback.py?user=Ainali The API is taking ~30 secs to respond. @ashishkumar468 Can you check the timeout for our API calls. Maybe we can increase the API timeout. @urbanecm Is there any way to cache the responses or reduce the API latency? |
I'll have a look on optimalizing the queries. |
@urbanecm Have created an issue in the repo about one quick optimization that can be done. |
As part of optimising the queries, I have submitted the following PRs. commons-app/commonsmisc#5 |
* Added a snack with retry when api in AA fails * Increased connection timeouts in okhttpclient builder
Hi all, I've looked on @maskaravivek 's pull requests and merged some of them. commons-app/commonsmisc#5 looks to be the last one unmerged, as the suggested query looks to took even more (1 min). Also, I've reverted commons-app/commonsmisc#11, because of some issues I didn't spot before merging. See more in the pullrequest itself. PS: I apologize for not looking into this problem myself, thank you @maskaravivek for doing so instead of me. |
Summary:
From a user on our google group forum: "Tried to view my achievements, nothing shows and I briefly saw an error message."
System logs:
Device and Android version:
Android 8.0.0 on a Samsung galaxy S7 Edge
Commons app version:
2.9
Screen-shots:
Would you like to work on the issue?
Pref not
The text was updated successfully, but these errors were encountered: