SSWConsulting / SSWConsulting/SSW.Rewards.Mobile

💄 Only display "This week" on leaderboard if there is activity this week

Open
#556 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C#
Stars
37
Forks
11
PR merge metrics
No merged PRs in 30d

Description

Cc: @jeanthirion @tkapa @JackDevAU @tomek-i @ricksu978 @kahamason

Hi

Pain

As per #491 we now have "This week" filtering on the leaderboard and it is the default. On that issue, a few people commented that this could look bad when there is little or no traffic/volume/engagement.

Suggested Solution

There was one solution proposed on that thread to introduce levels. I think this bears investigation, as does a thorough analysis of what the best UX for this is that strikes the right balance between not looking like a wasteland while simultaneously not intimidating new users.

In the meantime, I propose this interim method. Which is to hide the "This week" tab if there are no points accrued by any users this week.

Tasks
  • Add logic to determine whether this week includes any points
  • Hide this week tab if there are no points
Acceptance Criteria
  • Meets Definition of Done

Thanks!

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the leaderboard implementation and the existing “This week” filtering introduced in issue #491. Inspect how weekly points are calculated, then determine how the tab can be conditionally displayed when no points exist; done means the tab is hidden with no weekly activity and remains available when points are present.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
mobile
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.