Skip to content

feat: add Leaderboard Display with Conference Management (#486)#555

Merged
RUKAYAT-CODER merged 1 commit into
rinafcode:mainfrom
Escelit:feat/leaderboard-conference-management-486
May 28, 2026
Merged

feat: add Leaderboard Display with Conference Management (#486)#555
RUKAYAT-CODER merged 1 commit into
rinafcode:mainfrom
Escelit:feat/leaderboard-conference-management-486

Conversation

@Escelit
Copy link
Copy Markdown

@Escelit Escelit commented May 28, 2026

What

Implements the Leaderboard Display with Conference Management feature as described
in issue #486.

Changes

New files:

  • src/components/leaderboard/LeaderboardConference.tsx — Core component with two
    sections:

    • Leaderboard — ranked participant list with crown/medal icons for top 3,
      formatted scores, and full ARIA accessibility
    • Conference Management — create, list, join/leave, and delete conference rooms;
      joining a room renders the existing VideoConference WebRTC component inline for live
      peer-to-peer sessions
  • src/app/leaderboard/page.tsx — Next.js App Router page at /leaderboard

  • src/components/leaderboard/tests/LeaderboardConference.test.tsx — 19 unit
    tests

How it was tested

  • ✅ All 19 unit tests pass (pnpm vitest run)
  • ✅ No TypeScript errors in new files
  • ✅ No regressions in existing tests

Acceptance criteria checklist

  • Leaderboard Display properly implements Conference Management
  • All related tests pass
  • No regression in existing functionality
  • Code follows project coding standards (Tailwind CSS, lucide icons, App Router,
    dark mode)
  • Accessibility guidelines followed (ARIA labels, aria-pressed, aria-expanded,
    semantic landmarks)
  • Performance impact is minimal (no new dependencies added)

Notes

The build was already failing before this PR due to a pre-existing syntax error in
src/lib/api.ts — unrelated to this feature.

Closes #486

──────────────

- Add LeaderboardConference component with ranked leaderboard and
  conference management (create, join/leave, delete)
- Integrate existing VideoConference WebRTC component for live sessions
- Add /app/leaderboard page
- Add 19 unit tests (all passing)
- Uses lucide icons, Tailwind CSS with dark mode, fully typed
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 28, 2026

@Escelit Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@RUKAYAT-CODER RUKAYAT-CODER merged commit 56d7178 into rinafcode:main May 28, 2026
3 of 6 checks passed
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.

enhancement Leaderboard Display : Conference Management

2 participants