Skip to content

docs: explain city rank - #109

Open
therunnas wants to merge 1 commit into
srizzon:mainfrom
therunnas:docs/explain-city-rank
Open

docs: explain city rank#109
therunnas wants to merge 1 commit into
srizzon:mainfrom
therunnas:docs/explain-city-rank

Conversation

@therunnas

@therunnas therunnas commented Jun 4, 2026

Copy link
Copy Markdown

What does this PR do?

Adds a short, shared explanation of what City Rank means and reuses it where rank is currently shown without enough context.

Changes:

  • add src/lib/city-rank.ts with shared rank copy
  • show the explanation under the rank badge on /dev/[username], with title and aria-label on the badge
  • clarify weekly and monthly digest labels/body text so email recipients understand rank is based on total GitHub contributions

Related issue

Closes #71

Screenshots

Local verification on /dev/dev_001:

Local screenshot showing the City Rank helper text

The Vercel preview currently requires maintainer authorization because this PR comes from a fork.

Checklist

  • npm run lint passes - full repo lint currently fails on pre-existing baseline errors outside this PR
  • Tested locally - scoped ESLint passes on the changed files and the profile page renders the helper text
  • No secrets or .env values committed

@vercel

vercel Bot commented Jun 4, 2026

Copy link
Copy Markdown

@therunnas is attempting to deploy a commit to the Samuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

[Bug] What is the Rank?

1 participant