Skip to content

Fix admin submission review flow for Goal 6#55

Open
kota1112 wants to merge 2 commits into
mainfrom
s2-k2
Open

Fix admin submission review flow for Goal 6#55
kota1112 wants to merge 2 commits into
mainfrom
s2-k2

Conversation

@kota1112
Copy link
Copy Markdown
Collaborator

Summary

Adjusted the admin submission review flow for Sprint 2 Goal 6.

Changes from main

  • Made Pending Submissions fetching tolerant of environment differences by handling both pending and pending_review
  • Saved reviewed_by and reviewed_at when an admin approves or rejects a submission
  • Made reject handling compatible across environments by falling back from rejected to denied when needed
  • Prevented a submissions query failure from causing the rest of the admin dashboard data to appear empty

Files changed

  • src/pages/AdminDashboard.tsx

Notes

  • This PR only changes the admin submission review flow
  • No final functional changes were kept outside src/pages/AdminDashboard.tsx

Remaining risk

  • Submission status values are still inconsistent across environments
  • Approve status update and XP update are still separate queries

@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
no-code-jam Ready Ready Preview, Comment May 13, 2026 1:36pm

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