Skip to content

Bug fix issue #2042 (Blank Achievements Activity) #2050

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

Conversation

ashishkumar468
Copy link
Collaborator

Description (required)
When the api call to fetch the achievements somehow fails, a blank screen shows up and user is left with no option to retry.
Fixes #2042 Achievements is blank for some users.

What changes did you make and why?

  • 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

Tests performed (required)

Tested {build variant, ProdDebug} on {One Plus 3T} with API level {27}.

Screenshots showing what changed (optional - for UI changes)
device-2018-11-29-230404

* 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
@codecov-io
Copy link

codecov-io commented Nov 29, 2018

Codecov Report

Merging #2050 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2050      +/-   ##
=========================================
- Coverage    4.05%   4.04%   -0.01%     
=========================================
  Files         225     225              
  Lines       11357   11374      +17     
  Branches     1048    1049       +1     
=========================================
  Hits          460     460              
- Misses      10863   10880      +17     
  Partials       34      34
Impacted Files Coverage Δ
.../main/java/fr/free/nrw/commons/utils/ViewUtil.java 0% <0%> (ø) ⬆️
...nrw/commons/achievements/AchievementsActivity.java 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c2addd9...b8fd204. Read the comment docs.

@misaochan
Copy link
Member

Hi @ashishkumar468 , I'm sorry that I didn't clarify. AFAIK for this user, Achievements NEVER loads (although I messaged him to double check). I was hoping we could look into that. His full logs are in the private repo, I only posted part of it - so if the username and error log is insufficient, we can look at the bigger picture.

@neslihanturan
Copy link
Collaborator

neslihanturan commented Nov 30, 2018

Thanks @ashishkumar468 ! It works perfect for users who had temporary problems with loading achievements. But I won't merge this since @misaochan requested a more complete solution for users who experience permanent issues with that.

@ashishkumar468
Copy link
Collaborator Author

Added a separate pr for the fix, needed base branch changes. Closing this one

@ashishkumar468 ashishkumar468 deleted the bugfix/blank_achievements_activity branch December 1, 2018 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants