Skip to content

fix(match): disputed status display in MatchDetailView (#546)#637

Open
pitah23 wants to merge 1 commit into
Arenax-gaming:mainfrom
pitah23:fix/546-match-disputed-status-display
Open

fix(match): disputed status display in MatchDetailView (#546)#637
pitah23 wants to merge 1 commit into
Arenax-gaming:mainfrom
pitah23:fix/546-match-disputed-status-display

Conversation

@pitah23

@pitah23 pitah23 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #546. MatchDetailView was rendering disputed matches identically to completed matches with no visual differentiation.

Changes

frontend/src/components/match/MatchDetailView.tsx

  • Added isDisputed flag derived from match.status === 'disputed'
  • Renders a prominent "Under Review" warning banner at the top of the view when the match is disputed
  • Banner includes:
    • Warning icon + "Under Review" heading
    • Explanatory copy telling players an admin is reviewing their scores
    • Dispute deadline pulled from match.disputeDeadline
    • Side-by-side submitted score comparison (player1 vs player2)
    • "Contact support" mailto link
  • Hides the "Report Issue" (submit score) action button while isDisputed is true
  • Imports AlertTriangle from lucide-react for the banner icon

Acceptance Criteria

Criterion Status
Disputed matches show a prominent "Under Review" banner with dispute deadline
Players see submitted scores side-by-side
"Contact support" link included in disputed state
"Submit score" action hidden while dispute is active

Testing

No existing tests broken. The change is purely additive UI logic gated on match.status === 'disputed'.

…renax-gaming#546)

- Add prominent 'Under Review' banner when match.status === 'disputed'
- Display both players' submitted scores side-by-side for easy comparison
- Show dispute deadline (disputeDeadline) inside the banner
- Include 'Contact support' mailto link in disputed state
- Hide 'Report Issue' (submit score) button while dispute is active
@pitah23 pitah23 requested a review from anonfedora as a code owner June 26, 2026 12:36
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

@pitah23 is attempting to deploy a commit to the paul joseph's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@pitah23 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

[frontend] - Match score report form does not handle "disputed" status display

1 participant