Skip to content

Add judge notes view popup per project for admin view#321

Merged
MichaelZhao21 merged 2 commits intohackutd:masterfrom
balebbae:pr-judge-notes
Mar 22, 2026
Merged

Add judge notes view popup per project for admin view#321
MichaelZhao21 merged 2 commits intohackutd:masterfrom
balebbae:pr-judge-notes

Conversation

@balebbae
Copy link
Copy Markdown
Contributor

@balebbae balebbae commented Feb 23, 2026

Description

There's currently no way to see notes the judges write on the projects on the admin side. The "/judge/list" already pulls that data, I am just rendering it as a popup for admins.

UI Changes:
Screenshot 2026-02-20 at 3 27 14 PM
Screenshot 2026-02-20 at 3 27 20 PM

Fixes #316

Type of Change

Delete options that do not apply:

  • New feature (non-breaking change which adds functionality)

Is this a breaking change?

  • Yes
  • No

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 23, 2026

@balebbae is attempting to deploy a commit to the Michael Zhao's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 24, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
jury Ignored Ignored Preview Mar 22, 2026 1:39am

Copy link
Copy Markdown
Contributor

@MichaelZhao21 MichaelZhao21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some small code syntax things but looks good overall

@balebbae balebbae requested review from MichaelZhao21 and removed request for veermshah March 6, 2026 20:07
Copy link
Copy Markdown
Contributor

@MichaelZhao21 MichaelZhao21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@MichaelZhao21 MichaelZhao21 merged commit 96fc924 into hackutd:master Mar 22, 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.

Allow Admin to Read Judge Notes

2 participants