Skip to content

Leaderboard backend and frontend#26

Merged
biweep863 merged 20 commits intomainfrom
leaderboard-backend
Jun 18, 2025
Merged

Leaderboard backend and frontend#26
biweep863 merged 20 commits intomainfrom
leaderboard-backend

Conversation

@biweep863
Copy link
Copy Markdown
Contributor

  • Added leaderboard router
  • Implemented leaderboard UI
  • Added weekly leaderboard in the UI
  • FIx status variable on problem model
  • Fix UI for handling problem status based on the user.

@biweep863 biweep863 linked an issue Jun 11, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Member

@Oscar-gg Oscar-gg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice. See attached comments

Copy link
Copy Markdown
Member

@Oscar-gg Oscar-gg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, your branch wasn't building for me locally, so I added a workflow that builds the project in the main branch. Please make sure your pr builds before merging. (p.s., you need to make another commit to retrigger the updated workflow).

@biweep863 biweep863 requested a review from Oscar-gg June 17, 2025 01:29
Copy link
Copy Markdown
Member

@Oscar-gg Oscar-gg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, lgtm!! Check general comments and make sure to modify the classname concatenation using cn

@biweep863 biweep863 merged commit 94ad852 into main Jun 18, 2025
3 checks passed
@biweep863 biweep863 deleted the leaderboard-backend branch June 18, 2025 16:56
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.

US3.1 - Leaderboard backend

3 participants