Sriamsh - Implement “Project Status” Donut Chart#4495
Sriamsh - Implement “Project Status” Donut Chart#4495Sriamshreddy000 wants to merge 5 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
sayali-2308
left a comment
There was a problem hiding this comment.
- Branch: sriamsh_project_status_donut_chart_frontend
- Backend: Development branch
- Tested on: Windows, Chrome
- Date: February 9, 2026
What Works:
- Donut Chart Display: Chart renders correctly with three colored segments (Purple: Active Projects, Cyan: Completed Projects, Pink: Delayed Projects)
- Mock Data: Successfully displays mock data (Total: 50, Active: 20, Completed: 20, Delayed: 10)
- Summary Panel:** Right-side panel displays current date (Mon, Feb 9, 2026) and project counts with proper formatting
- Date Filters: Start date and end date input fields render correctly with proper format
- Apply Button: Blue "Apply" button displays and is clickable
- Center Label: "Total Projects: 50" displays correctly in donut center
- Legend: Chart legend shows all three categories with correct colors
- Dark Mode: Feature works perfectly in dark mode - all text readable, good contrast, chart visible
- Light Mode: Feature works perfectly in light mode
- No Console Errors: Console clean with no errors or warnings
Code Modification Required:
During testing, discovered that the mock data placement in the original code caused the component to display "Unable to load project status." error message.
Issue: The mock data was positioned AFTER the axios API call (lines 35-41), which meant the API call would execute first and potentially throw an error before reaching the mock data.
Solution Applied: Moved mock data block to immediately after setError(null) (before the API call) and added setLoading(false) before the return statement. This ensures mock data loads without triggering the error state.
Note:
As documented in PR description, backend returns zero counts in Dev environment, which is expected. Mock data was uncommented for testing purposes to verify chart functionality, colors, and layout as specified.
Feature works as described. Chart displays correctly, dark mode compatible, and integrates cleanly into BM Dashboard. Mock data code placement should be adjusted as noted above before final merge to prevent error messages during development testing
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Sriamsh,
I have reviewed your PR locally and was able to load the chart with the help of your mock data. However found the below issues:
- The Total Project is not in the center of the donut
- The chart is not adapted to dark mode
- The dates, i am able to select to date before the from data
- I edited the values to the one mentioned in the document and reloaded the page, however there is no difference in the % length of the different colored semi circle edges
- The label are very close to the chart with no spacing
- The chart is not mobile responsive
Hi Anusha, I’ve pushed updates addressing the feedback (center alignment, responsiveness, legend spacing, date validation, and percentage rendering). |
Hi Rohan, |
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Sriamsh,
Thanks for fixing the issues. Do mention in your description that you have not implemented backend with the reason. One more suggestion that i forgot to mention was the no data warning message, it would be great if you could center it. However everything else works hence approving the PR.
one-community
left a comment
There was a problem hiding this comment.
Anusha's requests in her last comment need to be addressed
|
I have added a note in the PR description clarifying the backend changes. I have also centered the “No project status data available.” message as suggested. |









Description
Related PRS (if any):
This frontend PR is related to the development backend PR.
To test this frontend PR you need to checkout the development backend PR.
Main changes explained:
How to test:
npm installandnpm startto run this PR locallyScreenshots or videos of changes:
Note: