Skip to content

Ramsundar - Prevent duplicate PR entries in PR Grading Dashboard (backend)#2077

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

Ramsundar - Prevent duplicate PR entries in PR Grading Dashboard (backend)#2077
Ram-blip wants to merge 1 commit intodevelopmentfrom
Prevent_Duplicate_PR_entries_for_PR_grading_dashboard_Backend

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 backend PR is related to the #4921 frontend PR.

Main changes explained:

  • Updated normalizePrNumbers to strip all whitespace for consistent key comparison
  • Integrated this check into validateGradingEntry as a defensive layer before the data reaches mergeGradedPrs

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.

@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