Skip to content

Commit aeb9d44

Browse files
authored
Merge pull request #76 from crux-bphc/fix/leaderboard-year-sort-order
Fixed a bug where the years would not be sorted in the batch dropdown
2 parents a1e5636 + 4482e2c commit aeb9d44

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

frontend/src/routes/leaderboard/$slug.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@ function RouteComponent() {
5252

5353
const batches = [
5454
...new Set<string>(leaderboardData.map((user) => user.batch)),
55-
].filter((batch) => (batch ? true : false));
55+
]
56+
.filter((batch) => (batch ? true : false))
57+
.sort((a, b) => parseInt(b) - parseInt(a));
5658

5759
if (loading) return <LoadingIndicator />;
5860

frontend/src/routes/leaderboard/index.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,9 @@ function RouteComponent() {
5151

5252
const batches = [
5353
...new Set<string>(leaderboardData.map((user) => user.batch)),
54-
].filter((batch) => (batch ? true : false));
54+
]
55+
.filter((batch) => (batch ? true : false))
56+
.sort((a, b) => parseInt(b) - parseInt(a));
5557

5658
if (loading) return <LoadingIndicator />;
5759

0 commit comments

Comments
 (0)