Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hacker Leaderboard and Judging Schedule #399

Merged
merged 15 commits into from
Oct 15, 2023
Merged

Hacker Leaderboard and Judging Schedule #399

merged 15 commits into from
Oct 15, 2023

Conversation

carolhe134
Copy link
Contributor

@carolhe134 carolhe134 commented Oct 12, 2023

What I've done

HackerDash.tsx

  • Added Leaderboard and JudgingSchedule components

JudgingSchedule.tsx

  • Created component for judging schedule

Leaderboard.tsx

  • Created component for leaderboard
  • Gets data from /api/leaderboard
  • Automatically updates leaderboard using SWR

/api/leaderboard

  • Created new endpoint for querying the top 10 hackers with the most NFC points

Leaderboard Screenshot

image

Judging Schedule Screenshots

When there's no judging schedule available

image

When there's a judging schedule available (dummy data not included in this PR)

image

@vercel
Copy link

vercel bot commented Oct 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
witness ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 13, 2023 3:23pm

@carolhe134 carolhe134 requested a review from zineanteoh October 12, 2023 06:10
@zineanteoh
Copy link
Member

@carolhe134 Is this ready for review?

@carolhe134
Copy link
Contributor Author

@carolhe134 Is this ready for review?

Yes @zineanteoh

Copy link
Member

@zineanteoh zineanteoh left a comment

Choose a reason for hiding this comment

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

LGTM.
@carolhe134 nice work! 💯

@zineanteoh zineanteoh merged commit 889b0fc into main Oct 15, 2023
5 checks passed
@zineanteoh zineanteoh deleted the hacker-css-carol branch October 15, 2023 02:47
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.

2 participants