Skip to content

Fix for submissions tab with > 500 submissions on a challenge (for MM)#7207

Merged
jmgasper merged 4 commits into
masterfrom
develop
Apr 21, 2026
Merged

Fix for submissions tab with > 500 submissions on a challenge (for MM)#7207
jmgasper merged 4 commits into
masterfrom
develop

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

No description provided.

jmgasper added 4 commits April 4, 2026 05:55
What was broken
Marathon Match final scores stayed hidden in Community App while the review phase was still open, even after Review API had stored the system-test final scores.

Root cause
The MM submission views treated review completion as a prerequisite for rendering final scores, so open-review challenges forced the final-score fields to `N/A` or `-` despite having `finalScore` data.

What was changed
Removed the review-complete gate from the MM submissions table and submission history rows.
Updated My Submissions and the submission details modal to surface `finalScore` as soon as it exists, while still preferring `initialScore` over stale provisional-score values.

Any added/updated tests
Updated the My Submissions score regression test to verify final scores remain visible before the review phase closes.
# Conflicts:
#	__tests__/shared/components/challenge-detail/MySubmissions/SubmissionsList/index.jsx
#	src/shared/components/challenge-detail/MySubmissions/SubmissionsList/index.jsx
#	src/shared/components/challenge-detail/Submissions/SubmissionInformationModal/index.jsx
#	src/shared/components/challenge-detail/Submissions/SubmissionRow/SubmissionHistoryRow/index.jsx
#	src/shared/components/challenge-detail/Submissions/SubmissionRow/index.jsx
PM-4608: show MM final scores during review
@jmgasper jmgasper requested a review from kkartunov as a code owner April 21, 2026 05:03
@jmgasper jmgasper merged commit ada531c into master Apr 21, 2026
9 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