Skip to content

feat(reviews): add Priority badge for applications submitted by Oct 3 11:59pm CT - #182

Merged
balebbae merged 1 commit into
mainfrom
devin/1790995178-priority-badge
Oct 3, 2026
Merged

balebbae merged 1 commit into
mainfrom
devin/1790995178-priority-badge

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Frontend-only: reviewers can now see which applications were submitted before the priority deadline.

  • New pages/admin/_shared/priority.ts owns the cutoff and predicate:
    export const PRIORITY_DEADLINE = new Date("2026-10-03T23:59:59.999-05:00"); // Oct 3 11:59pm CDT
    export function isPriorityApplication(submittedAt): boolean // false for null/invalid/after cutoff
  • New PriorityBadge (_shared/PriorityBadge.tsx) renders a purple Zap + "Priority" badge, or null when !isPriorityApplication(submittedAt). Tooltip shows the cutoff in CT.
  • Badge placed next to the existing status/vote badge in:
    • super admin: reviews/components/ReviewsTable.tsx (rows use ApplicationListItem.submitted_at), reviews/grading/GradingPage.tsx header
    • admin: reviews/grading/GradingPage.tsx header, ReviewsPage.tsx completed-review sheet, all-applicants table + ApplicationDetailPanel header, and TimelineSection "Submitted" line (which also shows in the grading details panel)

Not covered: the admin Reviews list rows (Review type) don't carry submitted_at, so the badge only appears once a review is opened — adding it to the list would need an API change.

Cutoff timezone is assumed Central; change the -05:00 offset in priority.ts if that's wrong.

Screenshots

Super admin reviews table (Riley/Morgan submitted after cutoff, no badge):
superadmin table

Super admin grading:
superadmin grading

Admin grading:
admin grading

Link to Devin session: https://app.devin.ai/sessions/f23c7daefd9e4c57834ef8cf6bf28d8e
Open in Devin Desktop: https://app.devin.ai/desktop/session/f23c7daefd9e4c57834ef8cf6bf28d8e?variant=devin
Requested by: @balebbae

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@balebbae
balebbae enabled auto-merge (squash) October 3, 2026 02:41
@balebbae
balebbae disabled auto-merge October 3, 2026 02:43
@balebbae
balebbae merged commit 088e90d into main Oct 3, 2026
3 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