Skip to content

Neeraj Fix Chart Rendering Issue and Dark Mode Legend#5123

Open
Neeraj-Kondaveeti wants to merge 1 commit intodevelopmentfrom
Neeraj_Hotfix_HiringAnalytics_Chart
Open

Neeraj Fix Chart Rendering Issue and Dark Mode Legend#5123
Neeraj-Kondaveeti wants to merge 1 commit intodevelopmentfrom
Neeraj_Hotfix_HiringAnalytics_Chart

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Apr 9, 2026

Description

INITIAL TASK DESCRIPTION:

image

CHANGES REQUESTED:
image

Related PRS (if any):

This frontend PR is related to the Development Backend PR.

This PR is hotfix for #4896

Main changes explained:

  • Fixed an issue where the Hiring Analytics chart was not rendering.
  • Updated API filter handling to allow fetching all roles when no roles are selected.
  • Fixed dark mode legend display by replacing text-based indicators with styled color blocks to correctly match chart bars.

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. Navigate to: http://localhost:5173/applicant-volunteer-ratio
  6. Verify:
    • Chart loads correctly and does not get stuck on "Loading..."
  7. Toggle dark mode:
    • Legend colors should match chart bars (blue for Total Applications, green for People Hired)
    • Legend should be clearly visible and readable

Screenshots or videos of changes:

Screenshot 2026-04-09 at 12 26 45 AM Screenshot 2026-04-09 at 12 26 56 AM

Note:

  • This PR specifically addresses post-merge feedback related to chart rendering and dark mode legend visibility

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 0fe1ea1
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69d72bcaaa91fd000818aad3
😎 Deploy Preview https://deploy-preview-5123--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.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 9, 2026

@Neeraj-Kondaveeti Neeraj-Kondaveeti added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 9, 2026
@rajanidi1999
Copy link
Copy Markdown

image working as expected and im approving the PR

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.

2 participants