Skip to content

Pranav - fix(backend): create donut chart for applicant experience breakdown#1606

Merged
one-community merged 14 commits into
developmentfrom
pranav-create-donutChart
Sep 14, 2025
Merged

Pranav - fix(backend): create donut chart for applicant experience breakdown#1606
one-community merged 14 commits into
developmentfrom
pranav-create-donutChart

Conversation

@pranavgovi
Copy link
Copy Markdown
Contributor

@pranavgovi pranavgovi commented Jul 27, 2025

Description

Screenshot 2025-07-26 223107

Related PRS (if any):

This is related to PR#3840

Main changes explained:

  • Updated jobApplicants model
  • Updated app.js to include applicants route
  • Updated applicant analytics controller

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. go to postman and obtain a valid token by sending a POST request to http://localhost:4500/api/login
  6. Use the token and send a GET request to http://localhost:4500/api/applicants/experience-breakdown to obtain the breakdown of applicants based on experience.
  7. Also test it by sending start date and end date params like http://localhost:4500/api/applicants/experience-breakdown?startDate=2020-01-01&endDate=2025-01-01

Screenshots or videos of changes:

Note:

This backend PR needs some work regarding filtering by role. So don't test it for now. Front end also should be completed by next week.

Copy link
Copy Markdown
Contributor

@csk731 csk731 left a comment

Choose a reason for hiding this comment

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

The changes look good to me, and the API response is working as expected.

Also, when the startDate is greater than the endDate, the response is empty, which is a good edge case handling.

Screenshot 2025-08-01 at 6 16 07 PM Screenshot 2025-08-01 at 6 16 55 PM Screenshot 2025-08-01 at 6 17 33 PM Screenshot 2025-08-01 at 6 16 07 PM Screenshot 2025-08-01 at 6 16 55 PM Screenshot 2025-08-01 at 6 17 33 PM Screenshot 2025-08-01 at 6 20 50 PM

@csk731 csk731 self-requested a review August 1, 2025 22:22
@suparshwa31
Copy link
Copy Markdown
Contributor

I tested the changes and both the api endpoints are working great.
Screenshot 2025-08-02 at 17 25 40
Screenshot 2025-08-02 at 17 25 50

Copy link
Copy Markdown
Contributor

@sirisudheeksha sirisudheeksha left a comment

Choose a reason for hiding this comment

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

Tested the changes locally both the API endpoints are working as expected and returning correct experience breakdown data.
image

image

Copy link
Copy Markdown
Contributor

@beblicarl beblicarl left a comment

Choose a reason for hiding this comment

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

Reviewed the frontend and everything worked as intended and the code is great

OneCommunityGlobal/HighestGoodNetworkApp#3840 (comment)

@one-community one-community changed the title fix(backend): create donut chart for applicant experience breakdown Pranav - fix(backend): create donut chart for applicant experience breakdown Sep 13, 2025
@one-community one-community merged commit 94a522f into development Sep 14, 2025
1 check passed
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.

6 participants