Skip to content

Prod - Getting ready for beta marathon match - #260

Merged
jmgasper merged 5 commits into
masterfrom
develop
May 22, 2026
Merged

Prod - Getting ready for beta marathon match#260
jmgasper merged 5 commits into
masterfrom
develop

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

No description provided.

jmgasper added 5 commits May 18, 2026 15:42
What was broken
The My Active Challenges review endpoint did not return the number of challenge submissions or whether Iterative Review was currently open, so the review app could not flag Topgear Task challenges where submitted work was waiting but Iterative Review had not opened.

Root cause (if identifiable)
The my-reviews summary query only selected review progress and current phase details; it did not include the challenge submission count or an explicit Iterative Review phase-open signal.

What was changed
Added numOfSubmissions and isIterativeReviewPhaseOpen to the my-reviews DTO and summary query, including active and past query paths.

Any added/updated tests
Added service coverage that verifies the new fields are selected and returned for active review challenges.
PM-5059: Expose active review submission state
What was broken
Registered Marathon Match submitters could only retrieve their own review summation rows, so the Community app could not show other competitors' summation scores before challenge completion.

Root cause
The review summation search path validated submitter access for Marathon Match challenges, then still added the requester member id to the submission filter.

What was changed
Removed the submitter member-id filter after Marathon Match registration/type validation, allowing registered competitors to fetch challenge-level review summations while preserving the existing metadata sanitization.

Any added/updated tests
Updated the review summation service unit test to cover multiple submitters' summations and verify per-seed metadata remains hidden.
PM-5111: Allow marathon submitters to view scores
@jmgasper
jmgasper merged commit 05ae91f into master May 22, 2026
8 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