apps-android-commons icon indicating copy to clipboard operation
apps-android-commons copied to clipboard

Leaderboard: Taping on my avatar should open tip popup explaining how to change it

Open nicolas-raoul opened this issue 3 years ago • 14 comments

When tapping the round shown by the red arrow: Screenshot_20220131-120111_Commons

... show this tooltip:

To set up your leaderboard avatar, tap "Set as avatar" in the three-dots menu of any image."

nicolas-raoul avatar Jan 31 '22 03:01 nicolas-raoul

Hi, @nicolas-raoul I would like to look into this. Can you assign me?

ratneshnavlakhe avatar Feb 01 '22 13:02 ratneshnavlakhe

@ratneshnavlakhe It is yours, thanks! :-)

nicolas-raoul avatar Feb 01 '22 13:02 nicolas-raoul

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 avatar Feb 01 '22 18:02 ratneshnavlakhe

@ratneshnavlakhe Can you reproduce this problem on the master branch? If yes, please create a new GitHub issue, thanks for finding this! :-)

nicolas-raoul avatar Feb 02 '22 01:02 nicolas-raoul

@ratneshnavlakhe are you running betaDebug build variant (as it seems from the logs)? Please test the leaderboard with prodDebug instead, thanks. :)

misaochan avatar Feb 02 '22 06:02 misaochan

@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 avatar Feb 02 '22 07:02 madhurgupta10

@madhurgupta10 Great idea, issue created: #4784

nicolas-raoul avatar Feb 02 '22 08:02 nicolas-raoul

Yeah makes @madhurgupta10 to add a toast if the feature is unavailable for current variants. Thanks, @misaochan will try it out with prodDebug build.

ratneshnavlakhe avatar Feb 02 '22 12:02 ratneshnavlakhe

@nicolas-raoul can i add this feature ?

samarthasthan avatar Aug 31 '22 18:08 samarthasthan

@SamarthAsthan Sure, feel free to make a PR.

madhurgupta10 avatar Sep 01 '22 00:09 madhurgupta10

@madhurgupta10 it will be great if it will open that persons profile picture on taping on avatar ?

samarthasthan avatar Sep 01 '22 13:09 samarthasthan

@SamarthAsthan Sorry it was ambiguous, I rewrote the issue title and description to detail exactly what must be done. Thanks! :-)

nicolas-raoul avatar Sep 01 '22 13:09 nicolas-raoul

@nicolas-raoul you wanna say like like instagram?

https://user-images.githubusercontent.com/97031010/187924514-dcb976be-3086-4d02-81b1-ba9a8ad7267d.MP4

samarthasthan avatar Sep 01 '22 13:09 samarthasthan

@SamarthAsthan Yes except it should show an explanation popup rather than a menu.

nicolas-raoul avatar Sep 01 '22 13:09 nicolas-raoul

Hi @nicolas-raoul , Is anyone working on this issue? Can I take this up?

bhavanagarlapati avatar Dec 06 '22 08:12 bhavanagarlapati

@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. :-)

nicolas-raoul avatar Dec 07 '22 08:12 nicolas-raoul

@SamarthAsthan Sorry I realize I did not respond. A simple toast or snackbar is enough. No buttons needed, just a label.

nicolas-raoul avatar Dec 07 '22 08:12 nicolas-raoul

@nicolas-raoul can you please assign me this issue.

ArmanNavodia avatar Feb 28 '23 17:02 ArmanNavodia

@ArmanNavodia It is yours, thanks! 🙂

nicolas-raoul avatar Mar 01 '23 00:03 nicolas-raoul