Skip to content

Neeraj Implement PR Count Consistency, Freezing Logic, And Dark Mode Support#4851

Open
Neeraj-Kondaveeti wants to merge 2 commits intodevelopmentfrom
Neeraj_PRAnalytics_PRCount_Fix
Open

Neeraj Implement PR Count Consistency, Freezing Logic, And Dark Mode Support#4851
Neeraj-Kondaveeti wants to merge 2 commits intodevelopmentfrom
Neeraj_PRAnalytics_PRCount_Fix

Conversation

@Neeraj-Kondaveeti
Copy link
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Feb 17, 2026

Description

image image

Related PRS (if any):

This frontend PR is related to the Development backend PR.

Main changes explained:

  • PR Count Consistency
    • PR reviewed now always equals gradedPrs.length
    • Prevents mismatch between count and actual PR list
  • Freeze Logic
    • Reviewed PR count cannot be manually reduced
    • Count is derived directly from graded PRs
    • Prevents altering previously reviewed PRs
  • Dark Mode Logic
    • Properly applied dark-mode class using Redux theme state
    • Table, inputs, modal, buttons now respond correctly

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. Navigate to: http://localhost:5173/pr-grading-screen
  6. Verify:
    • Adding PR updates count automatically
    • Count cannot be manually reduced
    • Modal opens & grading works
    • Dark mode styling applies correctly

Screenshots or videos of changes:

image image image image image

Note:

Changes are frontend only, no backend changes.

@netlify
Copy link

netlify bot commented Feb 17, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit bd8ac3d
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6994b8d27ee4040008d28851
😎 Deploy Preview https://deploy-preview-4851--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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments