Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025

The contest results page was mixing teacher/admin management functionality with participant-facing results, causing confusion about data visibility and access control.

Changes

Teacher/Admin Results (/dashboard/teacher/contests/[contestId]/results)

  • New page showing full leaderboard with all participant names and scores
  • Fetches aggregated stats from /contests-management/contests/{id}/user-stats
  • Displays contest-wide metrics (total participants, submissions, tasks solved, avg score)
  • Access controlled via existing contest collaborator validation

Participant Results (/dashboard/user/contests/[contestId]/results)

  • Removed leaderboard showing other participants
  • Only displays current user's results via /contests/{contestId}/results/my
  • Shows personal score, task completion, and submission breakdown

Remote Query Changes

// Teacher: fetches all participant stats
const [userStats, contest] = await Promise.all([
  contestsManagementService.getUserStats(params.contestId),
  contestService.getContest(params.contestId)
]);

// Participant: fetches only personal results
const myResults = await contestService.getMyResults(params.contestId);

Translations

Added teacher-specific i18n keys (teacher_contest_results_*) in English and Polish.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cdn.jsdelivr.net
    • Triggering command: /opt/hostedtoolcache/node/22.21.1/x64/bin/node node /home/REDACTED/work/frontend/frontend/node_modules/.bin/../vite/bin/vite.js build (dns block)
  • eu.posthog.com
    • Triggering command: /opt/hostedtoolcache/node/22.21.1/x64/bin/node node /home/REDACTED/work/frontend/frontend/node_modules/.bin/../vite/bin/vite.js build (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Contests Result page</issue_title>
<issue_description>Current implementation of contest results is a complete implmenetation mess. What we want is:

  • Contest results page for only teachers/admins which looks the same as current /user/contests/results page but is only accessable for contest collaborators. Endpoint should be /contests-management/contests/ one's which are only available for manager,s there is differnet ypes of aggregates returned.
  • Contest results page for participants, which should only return Results for user using /contests/id/results/my endpoint results of only curreny user.
  • In the future we will implement page for users but with all results available with redacted names of other participants.
    For now only first and second types of pages should be implmeneted. </issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 15, 2025 16:17
Co-authored-by: HermanPlay <78978614+HermanPlay@users.noreply.github.com>
Co-authored-by: HermanPlay <78978614+HermanPlay@users.noreply.github.com>
Copilot AI changed the title [WIP] Add contest results page for teachers and participants Split contest results into teacher management and participant views Dec 15, 2025
Copilot AI requested a review from HermanPlay December 15, 2025 16:25
TheRealSeber
TheRealSeber previously approved these changes Dec 15, 2025
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.

Contests Result page

3 participants