Skip to content

Shashank madan pr grading screen backend#1947

Merged
one-community merged 4 commits intodevelopmentfrom
shashank-madan-pr-grading-screen-backend
Jan 26, 2026
Merged

Shashank madan pr grading screen backend#1947
one-community merged 4 commits intodevelopmentfrom
shashank-madan-pr-grading-screen-backend

Conversation

@shashank-madan
Copy link
Copy Markdown
Contributor

Description

image

Related PRS (if any):

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

Main changes explained:

  • Added Pages for PR Grading dashboard
  • Implemented logic to allow only valid pr numbers as inputs
  • Added dark mode styling to all pages in pr grading dashboard

How to test:

  1. check into current branch
  2. do npm install, npm run build and npm start to run this PR locally
  3. Clear site data/cache
  4. log as owner user
  5. go to http://localhost:5173/pr-grading-dashboard
  6. Add a new reviewed PR to one of the reviewers
  7. Add a new reviewer
  8. Click Save
  9. Verify that you get toast messages indicating success at the steps.
  10. Refresh the page and verify that your changes have persisted
  11. Check dark mode as well

Screenshots or videos of changes:

PR-grading-dashboard.mp4

@shashank-madan shashank-madan added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Dec 9, 2025
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Shashank,

I have reviewed your PR locally and the backend works as per requirement.

@beblicarl
Copy link
Copy Markdown
Contributor

I have tested the frontend of this and it worked as intended and hence approved of it

OneCommunityGlobal/HighestGoodNetworkApp#4533 (comment)

@beblicarl beblicarl self-requested a review January 23, 2026 14:34
@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit f27be25 into development Jan 26, 2026
2 checks passed
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.

4 participants