Skip to content

Conversation

@Amalesh-A
Copy link
Contributor

@Amalesh-A Amalesh-A commented Jan 10, 2026

Description

image

Related PRS (if any):

This frontend PR is related to the development branch of the backend.
Related PR: #4548

Main changes explained:

  • Updated 'HoursCompletedBarChart.jsx'
  • Updated 'VolunteerTrendsLineChart.jsx'
  • Updated 'AnniversaryCelebrated.jsx'
  • Updated 'TeamStats.jsx'

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 an user
  5. go to dashboard→ Total Org Summary
  6. Check if the above bugs are resolved

Screenshots or videos of changes:

Before:
https://github.com/user-attachments/assets/ea2bb9a9-71ca-490b-99c5-86bed108ef86

After:
https://github.com/user-attachments/assets/2742e69b-5e2e-4074-a86d-534dcff22b20

@netlify
Copy link

netlify bot commented Jan 10, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 478ced0
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69630b4b29d01900081e9041
😎 Deploy Preview https://deploy-preview-4694--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Amalesh-A Amalesh-A changed the title Bugfixes Amalesh - PR4548 Bugfixes Jan 10, 2026
@sonarqubecloud
Copy link

@Amalesh-A Amalesh-A added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jan 13, 2026
Copy link

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

Well done with this implementation. A couple issues are the bar graph overlays are over-sized. Also the blue square has black text (see image below.)

PR 4694 Screenshot PR 4694 Screenshot 2

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.

3 participants