Skip to content

Alisha: (fix) Job Analytics Page - Add Job-Level Analytics Filter and Breakdown #4886

Open
alishawalunj wants to merge 13 commits intodevelopmentfrom
alisha/fix/job-level-analytics-filter-breakdown
Open

Alisha: (fix) Job Analytics Page - Add Job-Level Analytics Filter and Breakdown #4886
alishawalunj wants to merge 13 commits intodevelopmentfrom
alisha/fix/job-level-analytics-filter-breakdown

Conversation

@alishawalunj
Copy link
Copy Markdown
Contributor

@alishawalunj alishawalunj commented Feb 22, 2026

Description

F9714799-C9EB-470D-9DF2-656D30D67556

Related PRS (if any):

This frontend PR is not related to any backend PR. Checkout latest development branch for backend.

Main changes explained:

  • Updated file JobAnalytics to include dropdown for job role.
  • Updated all charts to reflect on Job role selection.
  • Fixed Pie chart, Bar chart and summary tiles to reflect values on applying filter.
  • Updated refresh button to set filter values to default values.

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 http://localhost:5173/application/analytics
  6. Verify below requirements:
  • A job selection dropdown is added to the dashboard header
  • Dropdown includes a mock list of jobs and an “All Jobs” option
  • Selecting a job updates all charts and summary metrics dynamically
  • Changing the date range or comparison period preserves the selected job
  • The UI clearly indicates the currently selected job
  • Refresh button still functions to reset all filters

Screenshots or videos of changes:

69C6E3FB-72CF-4F08-A102-A6434BDA6F12

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 22, 2026

Deploy Preview for highestgoodnetwork-dev ready!

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

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Mar 7, 2026
Copy link
Copy Markdown

@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 Alisha,

Can you review access rights?

Image

@alishawalunj alishawalunj force-pushed the alisha/fix/job-level-analytics-filter-breakdown branch from 53acce8 to c2f6a8f Compare March 7, 2026 18:38
@alishawalunj
Copy link
Copy Markdown
Contributor Author

Hi Alisha,

Can you review access rights?

Image

Hi rohan,

You need to login as dev admin user.

naznin07
naznin07 previously approved these changes Mar 11, 2026
Copy link
Copy Markdown

@naznin07 naznin07 left a comment

Choose a reason for hiding this comment

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

useAnalyticsData accepts selectedRole, making the hook reusable and scalable.

PR #4886

Anusha-Gali
Anusha-Gali previously approved these changes Mar 11, 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 Alisha,

I have reviewed your PR locally and it does work as per requirement.

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

I checked the current branch and ran the PR locally, but even after logging in as an admin user I am unable to access the analytics page. Because of this, I couldn’t verify the requirements. Please check once.

Image

Comment thread src/components/JobCCDashboard/JobAnalytics/JobAnalytics.jsx
Comment thread src/components/JobCCDashboard/JobAnalytics/JobAnalytics.jsx
@alishawalunj alishawalunj dismissed stale reviews from Anusha-Gali and naznin07 via 0596efe March 25, 2026 23:49
@alishawalunj alishawalunj force-pushed the alisha/fix/job-level-analytics-filter-breakdown branch from 20f2ddd to 0596efe Compare March 25, 2026 23:49
@alishawalunj
Copy link
Copy Markdown
Contributor Author

alishawalunj commented Mar 25, 2026

Hi Alisha,

I checked the current branch and ran the PR locally, but even after logging in as an admin user I am unable to access the analytics page. Because of this, I couldn’t verify the requirements. Please check once.

Image

Permissions have been updated. Could you please re-review?

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

I ran the PR locally again but was unable to access the analytics page even as an owner user, so I couldn't verify the requirements. The issue still persists. Please check once.

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

The permissions are still not reflected for owner accounts other than Dev Admin.

Image

@alishawalunj alishawalunj force-pushed the alisha/fix/job-level-analytics-filter-breakdown branch 2 times, most recently from e99e7c1 to c4fb16c Compare April 5, 2026 03:41
@alishawalunj
Copy link
Copy Markdown
Contributor Author

Image 4-5-26 at 12 07 AM

Fixed the issue regarding owner access.

Copy link
Copy Markdown

@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 Alisha,

Your changes seem to have not been properly pushed.

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

I have reviewed your PR locally and it works as per requirement assuming dark mode is not a part of requirement. Thanks for updating the permissions.

Image Image Image

@one-community
Copy link
Copy Markdown
Member

Hi Alisha,

I have reviewed your PR locally and it works as per requirement assuming dark mode is not a part of requirement. Thanks for updating the permissions.

Image Image Image

Thank you. Dark mode is always part of the requirement, please fix that Alisha.

@alishawalunj alishawalunj force-pushed the alisha/fix/job-level-analytics-filter-breakdown branch from 9106048 to b9ab8c7 Compare April 19, 2026 01:26
@sonarqubecloud
Copy link
Copy Markdown

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.

7 participants