Skip to content

Harshavarma: fix - Job Posting Analytics Graph Issues#5013

Open
harshavarma29 wants to merge 1 commit intodevelopmentfrom
Harshavarma_Fix_Job_Posting_Analytics_Graph
Open

Harshavarma: fix - Job Posting Analytics Graph Issues#5013
harshavarma29 wants to merge 1 commit intodevelopmentfrom
Harshavarma_Fix_Job_Posting_Analytics_Graph

Conversation

@harshavarma29
Copy link
Copy Markdown
Contributor

Description

Fixed the issues related to duplicate job roles, tooltip clarity, date range display in graph titles, and spacing between graphs, as reported in PR 4663

image

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  1. Consolidated duplicate job roles to ensure each role appears only once on the graph.
  2. Added the conversion rate formula to the tooltip for better clarity.
  3. Updated graph titles to reflect the selected date range (Yearly, Weekly, Monthly).
  4. Improved layout by adding spacing between the "Conversion Rate" and "Lowest Conversion Rate" graphs for better readability.

How to test:

  1. Check into 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 admin user
  5. Navigate to: http://localhost:5173/job-analytics

Verify the following

  1. Job roles appear uniquely on the y-axis.
  2. Selected date range is correctly displayed in graph titles.
  3. The conversion rate formula is visible in the tooltip.
  4. Enough space between graphs for clear visibility of graph labels and titles.
  5. Please check the attached test video for reference.

Screenshots or videos of changes:

Job_Analytics_Video_PR.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2026

Deploy Preview for highestgoodnetwork-dev ready!

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

Quality Gate Failed Quality Gate failed

Failed conditions
10.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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,

I checked this PR locally, and everything is working fine as expected according to the requirements. However, I noticed that the date range dropdown is not aligned properly in dark mode. Could you please confirm if this issue comes under your PR?

Image Image Image Image

Copy link
Copy Markdown

@Vishnu10-hub Vishnu10-hub left a comment

Choose a reason for hiding this comment

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

Hi,
Tested Harshavarma_Fix_Job_Posting_Analytics_Graph branch locally.
Job Analytics page loads correctly and graphs render without issues. No visual or functional regressions observed.
Feature is working as expected

Image Image 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,

I have reviewed your PR locally and the functionality does work as per requirement however since this is a PR for enhancing the page, could you also please look into:

  • Having some gap after the Date Range
  • Having some gap after the Show % checkbox and the text
  • And gap between Date Range and Show checkbox
  • The Date Range dropdown is not adapted to dark mode

Note: do let me know if you would not like to make those changes in this PR, i can approve as the functionality works

Image Image Image Image

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.

4 participants