Skip to content

Ramsundar - Prevent duplicate PR entries in PR Grading Dashboard (frontend)#4921

Open
Ram-blip wants to merge 1 commit intodevelopmentfrom
frontend-prevent-duplicate-pr-entries-pr-grading-dashboard
Open

Ramsundar - Prevent duplicate PR entries in PR Grading Dashboard (frontend)#4921
Ram-blip wants to merge 1 commit intodevelopmentfrom
frontend-prevent-duplicate-pr-entries-pr-grading-dashboard

Conversation

@Ram-blip
Copy link
Contributor

@Ram-blip Ram-blip commented Feb 28, 2026

Description

Screenshot 2026-02-28 at 7 23 37 PM

Related PRS (if any):

This frontend PR is related to the #2077 backend PR.

Main changes explained:

  • Pass the reviewer's existing graded PRs to the Add PR modal so it knows what's already been entered
  • Validate the new PR number against existing ones (ignoring whitespace differences like "1234+5678" vs "1234 + 5678")

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Go to /pr-grading-dashboard
  6. Click "Add new" button and check with existing Pr's for the particular user

Screenshots or videos of changes:

https://www.loom.com/share/cb7835414e18423f80d845daf0fcb0ba

Note:

Include the information the reviewers need to know.

@netlify
Copy link

netlify bot commented Feb 28, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit f73e038
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69a2f65c11c1a90008fe62a7
😎 Deploy Preview https://deploy-preview-4921--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants