Skip to content

Harshavarma: Created chart for job roles hit and applications#4325

Merged
one-community merged 7 commits intodevelopmentfrom
harsha_job_hits_and_applications
Apr 9, 2026
Merged

Harshavarma: Created chart for job roles hit and applications#4325
one-community merged 7 commits intodevelopmentfrom
harsha_job_hits_and_applications

Conversation

@harshavarma29
Copy link
Copy Markdown
Contributor

@harshavarma29 harshavarma29 commented Nov 4, 2025

Description

An analytics chart that displays the number of hits and applications for each role, with options to filter by date range and specific roles.
image

Related PRS (if any):

This frontend PR is related to the 1882 backend PR.

Main changes explained:

Created the following files:

  • src/actions/jobAnalytics/JobsHitsApplicationsActions.js: Fetches analytics data from the backend.
  • src/constants/jobAnalytics/JobsApplicationsHitsConstants.js: Contains constants for action dispatchers.
  • src/reducers/jobAnalytics/JobsHitsApplicationsReducer.js: Updates the state after a fetch, error, or loading action.
  • src/reducers/index.js: Configures the reducers.
  • src/components/JobAnalytics/JobsHitsApplicationsChart/JobsHitsApplicationsChart.jsx: Provides the UI for the application.
  • src/components/JobAnalytics/JobsHitsApplicationsChart/JobsHitsApplicationsChart.module.css: Defines CSS styles for the component.
  • src/routes.jsx: Configures the route for the Job Hits and Applications page.
  • src/utils/URL.js: Stores the backend endpoint as a constant.

How to test:

  1. Check into the current branch
  2. Do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. Log in as any user
  5. Go to http://localhost:5173/analytics/roles-hits-and-applications
  6. By default, the date and role filters should be set to “All”. Verify that the chart renders correctly.
  7. Verify the chart behavior when applying date and role filters individually and together.
  8. Verify that the page is responsive on smaller screens.
  9. Verify that the UI updates correctly in dark mode.
  10. Verify negative scenarios such as overlapping dates, invalid URLs, etc.
  11. Refer to the sample test video provided below.

Screenshots or videos of changes:

FE_Test_Instructions.mp4

Note:

Let me know your inputs, comments, or suggestions for improvement.

@netlify
Copy link
Copy Markdown

netlify bot commented Nov 4, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 8cd16db
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69d739a1c9d8ca0008a012c7
😎 Deploy Preview https://deploy-preview-4325--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 Nov 4, 2025

@harshavarma29 harshavarma29 changed the title Created chart for job roles hit and applications, added all the relev… Harsha - Created chart for job roles hit and applications, added all the relev… Nov 4, 2025
@harshavarma29 harshavarma29 changed the title Harsha - Created chart for job roles hit and applications, added all the relev… Harsha - Created chart for job roles hit and applications Nov 4, 2025
@harshavarma29 harshavarma29 changed the title Harsha - Created chart for job roles hit and applications Harsha: Created chart for job roles hit and applications Nov 4, 2025
@Anusha-Gali Anusha-Gali added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Nov 22, 2025
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Harsha,

I have reviewed your PR locally and though all the main functionalities work as per requriement and negative cases are handled, there is a small UI issue. As you can see below in both the modes that the names of roles are truncated.
Screenshot 2025-11-21 at 10 22 00 PM
Screenshot 2025-11-21 at 10 22 14 PM

Copy link
Copy Markdown
Contributor

@bhanuanishakkineni bhanuanishakkineni left a comment

Choose a reason for hiding this comment

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

Hello Harsha,
I have reviewed your PR and tested both frontend and backend. I have tested the charts behavior through selecting different dates and roles. The chart is working as expected and is responsive for all screen sizes. The UI is good in both light and dark modes.
One good thing I observed is that, if user selects an end date < start date, start date is automatically being changed to end date. One enhancement for this can be to alert users to select and end date later than start date.

PR.4325+.1882.testing-1.mp4

Please find comments related to backend in #1882

@harshavarma29 harshavarma29 changed the title Harsha: Created chart for job roles hit and applications Harshavarma: Created chart for job roles hit and applications Feb 23, 2026
Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hello Harsha,

I have reviewed your PR and tested the frontend. The functionality works as expected, but the Start Date, End Date, and Roles fields are not properly aligned in dark mode.

Image Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Harshavarma,

Thanks for making the changes, I have re-reviewed your PR, the only change i would suggest is ensuring full dark mode capability in start date and end date as well as roles text areas and also the roles dropdown and the toggle. Refer below image:
Image

Working screenshots
Image
Image
Image
Image
Image

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Harsha,

Thank you for making the improvements. I have reviewed your PR and tested the frontend. The Start Date, End Date, and Roles fields are now properly aligned in dark mode.

Image Image Image

Anusha-Gali
Anusha-Gali previously approved these changes Apr 5, 2026
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Harshavarma,

I have reviewed your PR locally and it works as per requirement. Thank you for fixing the issues.

Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 9, 2026

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 6ffad10 into development Apr 9, 2026
10 checks passed
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.

5 participants