Skip to content

fix: include top 3 ranks in leaderboard table - #2042

Open
ShreyaLbs wants to merge 1 commit into
gtech-mulearn:devfrom
ShreyaLbs:dev
Open

ShreyaLbs wants to merge 1 commit into
gtech-mulearn:devfrom
ShreyaLbs:dev

Conversation

@ShreyaLbs

@ShreyaLbs ShreyaLbs commented Apr 11, 2026 •

Copy link
Copy Markdown

Problem

The leaderboard table was skipping ranks 1, 2, and 3.
Users who wanted to see the full leaderboard in list format
could not do so because the top 3 were only shown in the
visual podium and completely absent from the table below.

Root Cause

In Leaderboard.tsx, the table was receiving
currentLeaderboard.slice(topPlayerCount) which cut off
the first 3 entries before passing data to LeaderboardTable.

Fix

Changed remainingPlayers to use the full currentLeaderboard
array so all ranks including 1, 2, and 3 appear in the table.

Changes

  • File: src/modules/Dashboard/modules/LeaderBoard/components/Leaderboard.tsx
  • Line 68: currentLeaderboard.slice(topPlayerCount) → currentLeaderboard

Steps to Reproduce

  1. Go to https://app.mulearn.org/dashboard/leaderboard
  2. Observe podium shows ranks 1, 2, 3
  3. Observe table starts from rank 4

Expected Behavior

Table should show all ranks starting from 1 so the
full leaderboard is visible in list format.

@vercel

vercel Bot commented Apr 11, 2026

Copy link
Copy Markdown

@ShreyaLbs is attempting to deploy a commit to the awindsr's projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

1 participant