Roshini Seelamsetty: Volunteer Hours Distribution Chart Shows Incorrect Totals and Misleading Visualization#4913
Open
roshini1212 wants to merge 21 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
HemanthNidamanuru
requested changes
Mar 16, 2026
Anusha-Gali
suggested changes
Mar 18, 2026
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Roshini,
I have reviewed your PR locally, though the main functionality works as per requirement have found the below issues:
- The chart data labels are not compatible to dark mode
- The chart data labels overlap in case of bigger numerical values
- The Total Hours Worked label is extended into to the graph
HemanthNidamanuru
previously approved these changes
Mar 20, 2026
I approved the changes accidentally.
HemanthNidamanuru
requested changes
Mar 20, 2026
rohanrastogi311
requested changes
Mar 21, 2026
552feb8 to
d70c366
Compare
added 19 commits
March 28, 2026 19:56
Changes: - Volunteer counts + % slices/legend - Center: Leaderboard total hours ✅ - Tooltips: 'Volunteers X, % Y' - Buckets: 40-49 → 50+ - Tests: 100% coverage + PropTypes - UX: formatRangeLabel(), guards
…, range label, and percentage calculation
…, range label, and percentage calculation
…Incorrect Totals and Misleading Visualization
d70c366 to
36aaa7b
Compare
HemanthNidamanuru
requested changes
Apr 7, 2026
Anusha-Gali
previously approved these changes
Apr 10, 2026
…chart-visualization-fix
…ding Visualization - Merge conflict
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


























Description
Fixes Yagna PRIORITY HIGH: Volunteer Hours Distribution Chart Shows Incorrect Totals and Misleading Visualization
Summary: Redesigned pie chart to show volunteer counts per hour range (not misleading hour values). Fixed center total, 40+/50+ buckets, tooltips, and percentage labels.
Main changes explained:
formatRangeLabel(),HoursWorkListw/ counts,userDataw/ percentagestotalHoursData.current(Leaderboard),tooltipType="hoursDistribution""X volunteers (Y%) logged Z+ hours..."How to test:
git checkout roshini/volunteer-hours-distribution-chart-visualization-fixin frontendnpm run start:localnpm test✅Screenshots:
BEFORE:

Screen.Recording.2026-02-26.at.15.12.04.mov
AFTER:
PR.4913.Light.Mode.mov
PR.4913.Dark.Mode.mov