Skip to content

[CU-86e1kwwx0]: Change GET /evaluations/courses/id/top-projects to se… - #138

Merged
WilmerJr01 merged 6 commits into
devfrom
CU-86e1kwwx0/Actualizar-endpoint-de-top-projects-para-permitir-cantidad-configurable
May 29, 2026
Merged

[CU-86e1kwwx0]: Change GET /evaluations/courses/id/top-projects to se…#138
WilmerJr01 merged 6 commits into
devfrom
CU-86e1kwwx0/Actualizar-endpoint-de-top-projects-para-permitir-cantidad-configurable

Conversation

@WilmerJr01

Copy link
Copy Markdown
Contributor

…lect n project in the top

@WilmerJr01
WilmerJr01 requested a review from a team May 28, 2026 22:05
@J-Povea21

Copy link
Copy Markdown
Collaborator

On GET /evaluations/courses/id/top-projects we are checking if Tiebreak if exist anyone register on it
On GET /evaluations/courses/id/top-projects return the Tiebreak if exists anyone that can change the current order of the TOP
On GET /evaluations/courses/id/top-projects return the Tiebreak if exists anyone that can change the current order of the TOP
On GET /evaluations/courses/id/top-projects return all disputed project on the next m positions that are fighting for a position in the top
On GET /evaluations/courses/id/top-projects return all disputed project on the next m positions that are fighting for a position in the top

@MrManini MrManini left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yay

@WilmerJr01
WilmerJr01 merged commit a8e1f67 into dev May 29, 2026
3 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.

3 participants