apps-android-commons
apps-android-commons copied to clipboard
Leaderboard: Taping on my avatar should open tip popup explaining how to change it
When tapping the round shown by the red arrow:

... show this tooltip:
To set up your leaderboard avatar, tap "Set as avatar" in the three-dots menu of any image."
Hi, @nicolas-raoul I would like to look into this. Can you assign me?
@ratneshnavlakhe It is yours, thanks! :-)
Hi @nicolas-raoul there seems to be some issue with loading leader board, getting this stack trace on logcat.
2022-02-01 23:49:51.214 8161-8161/fr.free.nrw.commons.beta E/AchievementsFragment: Fetching achievements statistics failed java.lang.StringIndexOutOfBoundsException: length=31; index=-1 at java.lang.String.substring(String.java:1935) at fr.free.nrw.commons.mwapi.OkHttpJsonApiClient.lambda$getAchievements$4$OkHttpJsonApiClient(OkHttpJsonApiClient.java:259) at fr.free.nrw.commons.mwapi.-$$Lambda$OkHttpJsonApiClient$-O4cqgnxRevLtbK8AVZXcOmTWMo.call(Unknown Source:6) at io.reactivex.internal.operators.single.SingleFromCallable.subscribeActual(SingleFromCallable.java:44) at io.reactivex.Single.subscribe(Single.java:3603) at io.reactivex.internal.operators.single.SingleSubscribeOn$SubscribeOnObserver.run(SingleSubscribeOn.java:89) at io.reactivex.Scheduler$DisposeTask.run(Scheduler.java:578) at io.reactivex.internal.schedulers.ScheduledRunnable.run(ScheduledRunnable.java:66) at io.reactivex.internal.schedulers.ScheduledRunnable.call(ScheduledRunnable.java:57) at java.util.concurrent.FutureTask.run(FutureTask.java:266) at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:301) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636) at java.lang.Thread.run(Thread.java:764) 2022-02-01 23:49:51.256 8161-8161/fr.free.nrw.commons.beta E/RecyclerView: No adapter attached; skipping layout
@ratneshnavlakhe Can you reproduce this problem on the master branch? If yes, please create a new GitHub issue, thanks for finding this! :-)
@ratneshnavlakhe are you running betaDebug build variant (as it seems from the logs)? Please test the leaderboard with prodDebug instead, thanks. :)
@ratneshnavlakhe are you running betaDebug build variant (as it seems from the logs)? Please test the leaderboard with prodDebug instead, thanks. :)
Maybe also adding a toast message saying "Leaderboard is only available on prod version" could be another good first issue.
@madhurgupta10 Great idea, issue created: #4784
Yeah makes @madhurgupta10 to add a toast if the feature is unavailable for current variants. Thanks, @misaochan will try it out with prodDebug build.
@nicolas-raoul can i add this feature ?
@SamarthAsthan Sure, feel free to make a PR.
@madhurgupta10 it will be great if it will open that persons profile picture on taping on avatar ?
@SamarthAsthan Sorry it was ambiguous, I rewrote the issue title and description to detail exactly what must be done. Thanks! :-)
@nicolas-raoul you wanna say like like instagram?
https://user-images.githubusercontent.com/97031010/187924514-dcb976be-3086-4d02-81b1-ba9a8ad7267d.MP4
@SamarthAsthan Yes except it should show an explanation popup rather than a menu.
Hi @nicolas-raoul , Is anyone working on this issue? Can I take this up?
@SamarthAsthan Any progress on this? :-)
@bhavanagarlapati Please take one issue at a time. Do not hesitate to come back to this one after sending a pull request for your currently assigned one. :-)
@SamarthAsthan Sorry I realize I did not respond. A simple toast or snackbar is enough. No buttons needed, just a label.
@nicolas-raoul can you please assign me this issue.
@ArmanNavodia It is yours, thanks! 🙂