Skip to content

Ramsundar - fix: prevent duplicate pr number entries on pr grading screen#4922

Open
Ram-blip wants to merge 1 commit intodevelopmentfrom
Prevent_Duplicate_PR_Entries_For_PR_grading_screen
Open

Ramsundar - fix: prevent duplicate pr number entries on pr grading screen#4922
Ram-blip wants to merge 1 commit intodevelopmentfrom
Prevent_Duplicate_PR_Entries_For_PR_grading_screen

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 development branch backend.

Main changes explained:

  • Before adding a PR, the screen now checks the current reviewer’s list (with normalized comparison) and shows an error and blocks the add if that PR number is already there; the error is shown in the UI and works in dark mode.

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-screen
  6. Click "Add new" button for each user and check with existing Pr's number for the particular user, an error line appears if you type an existing pr.

Screenshots or videos of changes:

Screenshot 2026-03-01 at 12 52 32 AM

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 4692d03
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69a3402291c5f60008fee483
😎 Deploy Preview https://deploy-preview-4922--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

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