Make achievements activity more discoverable

[vanshikaarora]

Achievements activity is currently accessed by clicking the icon. And that icon might not seem clickable to some general user

Why should we consider this?

• Presently there is no mention of achievements activity or any leaderboard in the app's tutorial
• Currently the icon is displayed over another image in the navigation drawer.
• To most general users the icon might not be percieved as a clickable element or it may be considered as a UI element.

What can we do?

• Since the learderboard is an activity. We can add it like another option in the navigation drawer
• Or if this icon looks significantly impressive to users then we can add the same icon to the toolbar

More suggestions on this opinion are highly appreciated :)

Would you like to work on the issue?
Yes

[nicolas-raoul]

While I agree that it is not visible enough now, I don't think adding another item to the navigation drawer is good, as it is already long. Displaying the current level of the user visible would certainly make it more attractive (for instance: Alaxandrea, level 7).

[vanshikaarora]

I don't think adding another item to the navigation drawer is good, as it is already long.

Accepted

How about adding the same icon to the toolbar?

Alternatively

Yes we can add the level of the user:
That adds a value. I can add a suggestion here how about adding this tutorial https://github.com/KeepSafe/TapTargetView for the first time when user uses this app.

[vanshikaarora]

@nicolas-raoul So what should i start with?

• Adding level number in place of icon
• Or Add the tutorial as mentioned in the previous link

I'll start working upon this soon afterwards

[nicolas-raoul]

I don't think it is important enough to add a tutorial step, a tutorial should only show features that are essential to get started.

Adding the level number means getting that number in the first place, which is not really easy, and would hurt performance if implemented incorrectly.

Since the achievement activity is currently broken, I don't think this is an issue we should focus on right now, so I suggest starting work on other tasks :-)

[vanshikaarora]

ok I'll see other tasks

[Tanvi-Goyal]

Hi i would like to work on this issue.

[nicolas-raoul]

@Tanvi-Goyal Assigned to you :-)

The current idea is to display the level number next to the user name.
Since the current achievements API is slow, this issue probably requires caching the level number (and only requesting it when it is absent or very old and also when the user enters the achievements activity.

[Tanvi-Goyal]

Can't we add the icon in the appbar left of the notiification icon.
Please consider else i will start working on the level number idea.
Thanks

[vanshikaarora]

Hey, If no-one is working on this Issue or shall I take this up?

[nicolas-raoul]

@Tanvi-Goyal That's an idea we can keep in mind, but I feel that near the username makes the user really feel that their achievements are associated to them. Did you get a chance to start caching the level or anything towards completing this issue? If yes please commit and push, even if it is not complete, thanks :-)

[ShashwatKedia]

@nicolas-raoul I would like to work on this issue, could you please tell me what are we finally gonna implement for this issue?

[nicolas-raoul]

Actually due to a UI revamp, the activity is now much more discoverable.

I suggest either:

1. closing this issue,
2. or changing the menu item from TheUser to TheUser's achievements. This must be internationalized.

@ShashwatKedia Do you want to try option 2?

[ShashwatKedia]

Sure, I would love to try that. @nicolas-raoul to confirm, if the username is Tester051005 with level 1, then in the menu, we want to show Tester051005 ( Level 1)

For caching the user level, I suggest storing the level in SharedPrefs.

Here right?

[ShashwatKedia]

@nicolas-raoul I tried to implement this and successfully stored the user level and displayed it in the menu item next to the username. But this works only after the user visits the achievements fragment once, since the data has to be fetched first; only then can I store it. So, when a user freshly installs and opens the app, the menu will display only their username but will show their level after visiting the achievements fragment once. Either we can go with this approach or close the issue since fetching data beforehand, just for the sake of showing level in the menu, doesn't seem feasible.

[nicolas-raoul]

Yes your screenshot shows the right place.

Tester051005 (Level 1) sounds good!

If the level is unknown yet, how about displaying Tester051005 (see your achievements)?

[ShashwatKedia]

If the level is unknown yet, how about displaying Tester051005 (see your achievements)?

Sounds great! It will make the user curious and tempt them to open it, nice. I'll do as required