Skip to content

feat: full-population Wall-of-Fame tabs (reviewers/issues/PR) + Top-50 cap#233

Merged
Progi1984 merged 3 commits into
PrestaShop:developfrom
PrestaEdit:feat/full-population-wof
Jul 1, 2026
Merged

feat: full-population Wall-of-Fame tabs (reviewers/issues/PR) + Top-50 cap#233
Progi1984 merged 3 commits into
PrestaShop:developfrom
PrestaEdit:feat/full-population-wof

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor

Follow-up to #231. Broadens the reviews/issues/PR leaderboards to the full population instead of only the top of each list.

Summary

  • Wall of Fame gains three tabs — Reviewers, Issue reporters, PR authors — listing everyone (committers and non-committers), searchable / sortable / paginated like the existing tabs. New type="ranking" on WallOfFameTable (avatar + name + login, count column, GitHub link, no modal).
  • Homepage Top section stays a curated highlight, now capped at 50 per leaderboard (slice(0, 50)); the full lists feed the Wall of Fame from the same app.vue refs.
  • Same RankingEntry contract — the data side just emits more entries.

Dependency

Needs the Traces change that emits the full lists with identity resolved at fetch time (no per-user API call): PrestaShop/traces#233. Once released and the scheduled Build & Deploy runs, the tabs populate with real full-population data.

Test Plan

  • vitest run — 9/9 green (ranking table, WoF tabs registered, Top-50 cap, empty-hide)
  • eslint . — clean
  • Visual check once traces#233 is released and data regenerates

🤖 Generated with Claude Code

@Progi1984
Progi1984 merged commit 74ba5e3 into PrestaShop:develop Jul 1, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for review to Merged in PR Dashboard Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants