Skip to content

PM-6302 Fix manual review status after AI failure -> dev - #2288

Merged
jmgasper merged 1 commit into
devfrom
fix/PM-6302-manual-review-status
Sep 13, 2026
Merged

PM-6302 Fix manual review status after AI failure -> dev#2288
jmgasper merged 1 commit into
devfrom
fix/PM-6302-manual-review-status

Conversation

@jmgasper

@jmgasper jmgasper commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator

Related JIRA Ticket:

PM-6302

What's in this PR?

Fixes QA's follow-up case where My Submissions still showed Ai Failed Review after the submission earned 100 in manual review. For AI_FAILED_REVIEW submissions, the latest final manual review's explicit pass/fail result now determines the current status: Completed for a pass, or Failed Review for a failure.

Submissions without a final manual outcome retain the AI failure. Other explicit submission statuses remain authoritative, and expanded AI workflow details retain the original AI result. Includes regression coverage and updated Opportunities documentation.

Validation

  • Confirmed the QA regression test fails against dev and passes with the fix.
  • All 101 challenge-detail tests pass.
  • Full lint and TypeScript checks pass.
  • Production build passes with unrelated source-map, CSS-ordering, and bundle-size warnings.

QA verification

Recheck the reported submission in My Submissions: its passing final manual review should show Completed alongside the score of 100. Expanding its AI details should still show the earlier failed AI result.

@jmgasper
jmgasper requested a review from kkartunov as a code owner September 13, 2026 01:05
@jmgasper
jmgasper merged commit 14e33d4 into dev Sep 13, 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