Skip to content

XXXX taking over for Manvitha phase4 project manager dashboard#1886

Open
myeeli wants to merge 8 commits into
developmentfrom
Manvitha-Phase4-ProjectManager-Dashboard
Open

XXXX taking over for Manvitha phase4 project manager dashboard#1886
myeeli wants to merge 8 commits into
developmentfrom
Manvitha-Phase4-ProjectManager-Dashboard

Conversation

@myeeli

@myeeli myeeli commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Description

image image

Related PRS (if any):

To test this backend PR you need to checkout the #4340 frontend PR.

Main changes explained:

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 localhost:5173/educationportal/pm/dashboard

Screenshots or videos of changes:

[

1.mp4

](url)

Note:

Include the information the reviewers need to know.

@Anusha-Gali Anusha-Gali left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Manvitha,

I have reviewed your PR locally and have left my comments in the frontend PR: OneCommunityGlobal/HighestGoodNetworkApp#4340 (review)

@VijayAnirudh VijayAnirudh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please refer to the comments made here: OneCommunityGlobal/HighestGoodNetworkApp#4340 (review)

@rapolusidhartha

Copy link
Copy Markdown
image

@rapolusidhartha rapolusidhartha left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Working as expected

@SharadhaKasiviswanathan
SharadhaKasiviswanathan dismissed stale reviews from Anusha-Gali and VijayAnirudh July 13, 2026 07:05

DISMISSED: The issues referenced in the linked frontend PR (HighestGoodNetworkApp#4340) have been fully resolved there.

Backend PR status:

  • Merge conflicts with development resolved (routes.js)
  • All lint checks passing (0 errors)
  • All tests passing (17 pre-existing integration test failures exist on development too - MongoDB env issue)
  • Ready for merge
@sonarqubecloud

Copy link
Copy Markdown

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

@kzou55 kzou55 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Sharadha,

I ran the PR and its associated frontend.

I couldn't verify the implementation as the frontend pr is giving me issues logging in.

@one-community one-community added Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. and removed High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible labels Jul 18, 2026
@one-community one-community changed the title Manvitha phase4 project manager dashboard XXXX taking over for Manvitha phase4 project manager dashboard Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants