Skip to content

[PROD] - Hotfixes#273

Open
kkartunov wants to merge 51 commits into
masterfrom
develop
Open

[PROD] - Hotfixes#273
kkartunov wants to merge 51 commits into
masterfrom
develop

Conversation

@kkartunov

Copy link
Copy Markdown
Contributor

No description provided.

vas3a and others added 30 commits June 2, 2026 09:30
…-submitters

PM-5226 - hide score for submitters
…er-platform/review-api-v6 into allow-to-edit-scores
jmgasper and others added 21 commits June 9, 2026 14:46
What was broken
Marathon Match scorer validation could not create a downloadable test submission before launch because Review API uploads either followed normal submission lifecycle checks or went to DMZ storage.

Root cause
The ECS scorer downloads clean submissions, while manual upload creates normal submissions through DMZ, phase, submitter, counter, scan, and notification side effects that are not appropriate for pre-launch validation.

What was changed
Added an admin/M2M validation-upload endpoint that stores the uploaded file directly in clean submission storage and creates an active file submission row without normal lifecycle side effects.

Any added/updated tests
Added unit tests for clean validation submission creation and missing-file rejection.
Consider timeout as terminal status, when fetching latest runs, move nulls to end
PM-5231: Add clean validation submission upload
Fix visibility for reviewer and MM for submissionId and score
Fixes for score visibility in MMs and challenges
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