Skip to content

Pause resubmission toggle + remove Airtable re-push button - #80

Merged
EDRipper merged 2 commits into
mainfrom
pause-resubmission
Aug 2, 2026
Merged

Pause resubmission toggle + remove Airtable re-push button#80
EDRipper merged 2 commits into
mainfrom
pause-resubmission

Conversation

@EDRipper

@EDRipper EDRipper commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • adds a Super-Admin toggle in the admin panel (Stats tab) to pause the changes-needed -> unreviewed resubmit path, for clearing the review queue
  • while paused: ProjectsService.update() rejects that transition with a 403, and the changes-needed slack DM (sent from first-pass review and fraud-review rejection) gets an extra note in Euan's words
  • new app_settings key/value table (migration 1785300000000-CreateAppSettings) backs the toggle, exposed via GET/POST /api/admin/settings/resubmission-paused (get = any reviewer, set = Super Admin only, audit-logged)
  • does not touch the separate "resubmit an already-approved project" flow (POST /projects/:id/resubmit), which is a voluntary re-ship, not the DM-driven changes-needed flow
  • removes the "Re-push to Airtable" button from the approved-project review panel (and its now-dead JS/CSS); backend endpoint left in place, just unwired from the UI

Test plan

  • npx tsc --noEmit (backend) clean
  • npm run build (backend) clean
  • npx jest (backend) - 93/93 passing, incl. updated admin-ban.spec.ts constructor mock
  • npm run check (frontend) - only pre-existing errors/warnings in files not touched by this PR
  • manually click the toggle in a deployed admin panel and confirm a changes-needed DM carries the note

EDRipper and others added 2 commits August 2, 2026 07:20
Lets a Super Admin pause the changes-needed -> unreviewed resubmit
path from the admin panel to clear the review queue, and appends
Euan's note to the changes-needed DM while paused.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@EDRipper
EDRipper merged commit 9606732 into main Aug 2, 2026
7 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