Skip to content

[CU-86e1mnu13]: add a new column - #150

Merged
isaAJ05 merged 2 commits into
devfrom
CU-86e1mnu13/Fixes-report-backend
Jun 6, 2026
Merged

[CU-86e1mnu13]: add a new column#150
isaAJ05 merged 2 commits into
devfrom
CU-86e1mnu13/Fixes-report-backend

Conversation

@isaAJ05

@isaAJ05 isaAJ05 commented May 31, 2026

Copy link
Copy Markdown
Contributor

Gateway: evaluation count: events.service.ts: generateRankingReport now fetches project stats and includes an evaluationCount field in the JSON output; the Excel export was updated to add an "Evaluaciones" column and populate it with the evaluation count.

@isaAJ05
isaAJ05 requested a review from a team May 31, 2026 00:48
@J-Povea21

Copy link
Copy Markdown
Collaborator

Task linked: CU-86e1mnu13 Fixes report backend

@isaAJ05
isaAJ05 requested a review from MrManini May 31, 2026 00:48

@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.

Ok on the new column but this endpoint is used somewhere else outside the ranking. Consider removing the filter and just changing that in the frontend side of it on rankings.

@isaAJ05
isaAJ05 requested a review from MrManini June 6, 2026 21:02
@isaAJ05 isaAJ05 changed the title [CU-86e1mnu13]: just approved projects and add a new column [CU-86e1mnu13]: add a new column Jun 6, 2026

@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.

Nice 👍

@isaAJ05
isaAJ05 merged commit f9de912 into dev Jun 6, 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