Skip to content

PM-5653 leaderboard calculations - #117

Merged
vas3a merged 2 commits into
developfrom
PM-5653_leaderboard-calculations
Aug 3, 2026
Merged

PM-5653 leaderboard calculations#117
vas3a merged 2 commits into
developfrom
PM-5653_leaderboard-calculations

Conversation

@vas3a

@vas3a vas3a commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

This pull request introduces new "leaderboard" reporting features for Topcoder challenges, including both generic and Marathon Match leaderboards. It adds new SQL report queries, DTOs for query validation, service types, and two new endpoints to expose this data. Additionally, it introduces a new scope for leaderboard reports and a development deployment script.

New leaderboard reporting features:

  • Added SQL queries for generic and Marathon Match leaderboards: leaderboard-generic.sql and leaderboard-mm.sql provide comprehensive challenge leaderboard data, including placements, scores, and prize information. [1] [2]
  • Introduced DTOs for leaderboard queries: LeaderboardGenericQueryDto and LeaderboardMmQueryDto enable robust query parameter validation and transformation for the new endpoints. [1] [2]
  • Added corresponding TypeScript types for leaderboard rows in the service layer to facilitate type-safe data handling.

API and access control enhancements:

  • Exposed two new endpoints in topcoder-reports.controller.ts for fetching leaderboard data: /topcoder/leaderboard/generic and /topcoder/leaderboard/mm, each with appropriate scope requirements.
  • Added a new scope, TopcoderLeaderboardReports, to app-constants.ts for fine-grained access control to leaderboard reports.

DevOps improvements:

  • Added a deploy:dev script in package.json for streamlined development deployments using branch-based tags.
  • Updated CircleCI config to trigger builds for tags matching dev-*, supporting the new deployment workflow.

@vas3a
vas3a merged commit 8c16f27 into develop Aug 3, 2026
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.

1 participant