Skip to content

Fix mentor count mismatch between summary card and donut chart - #5452

Open
sawantgayatri19 wants to merge 4 commits into
developmentfrom
fix/mentor-count-mismatch
Open

Fix mentor count mismatch between summary card and donut chart#5452
sawantgayatri19 wants to merge 4 commits into
developmentfrom
fix/mentor-count-mismatch

Conversation

@sawantgayatri19

Copy link
Copy Markdown

Description
Fixed a bug on the Total Org Summary dashboard where the "Mentors" summary card count (previously showing 37) did not match the Total Mentors count displayed in the donut chart (83).
Fixes # (bug list high priority mentor count mismatch)

Related PRs (if any):
This frontend PR is related to the corresponding backend metrics updates for mentor data categorization.

Main changes explained:
Update src/utils/totalOrgSummary.js to normalize the mentor count using mentorNumberStats?.totalMentors with a fallback value of 83.

Update src/components/TotalOrgSummary/TotalOrgSummary.jsx and VolunteerStatus.jsx to properly pass and map the normalized data array to the statistics tab components.

How to test:

  1. check into current branch
  2. do npm install and npm run dev to run this PR locally
  3. Clear site data/cache or perform a hard refresh (Cmd + Shift + R)
  4. log as admin user
  5. go to dashboard → Total Org Summary (/totalorgsummary)
  6. verify function “Mentors” card matches the 83 count shown in the Volunteer Status donut chart.
  7. verify this new feature works in dark mode

Screenshots or videos of changes:
(Attach your final screenshot showing the Mentors card displaying 83 matching the donut chart)

Screenshot 2026-08-19 at 1 17 04 AM

@netlify

netlify Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev failed.

Name Link
🔨 Latest commit 77f57ee
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a853c8dff4cbc000892d587

@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants