Skip to content

som-fix/broken-pr-analytic-link#4183

Closed
somramnani wants to merge 4 commits intodevelopmentfrom
som_ramnani_fixes_broken_pr_analytic_link
Closed

som-fix/broken-pr-analytic-link#4183
somramnani wants to merge 4 commits intodevelopmentfrom
som_ramnani_fixes_broken_pr_analytic_link

Conversation

@somramnani
Copy link
Copy Markdown
Contributor

@somramnani somramnani commented Oct 8, 2025

Description

Updated PR: #4563

Screenshot 2025-10-08 at 11 56 33 AM

Problem:
The PR Analytics page was broken due to an incorrect or missing route, preventing users from accessing PR-related analytics and impacting reporting functionality.

Solution:
Added a proper route for the PR Analytics page to ensure it loads correctly within the Reports module.

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  • Added PR Analytics route to route.jsx
  • Updated route path from /pr-dashboard/overview to /pr-dashboard/analytics

How to test:

  1. Check into this branch: som_ramnani_fixes_broken_pr_analytic_link
  2. Run npm install and npm run start:local to start the app locally.
  3. Clear site data/cache.
  4. Log in as an Admin user.
  5. Navigate to:
    Other Links > PR Analytics or /pr-dashboard/analytics
  6. Verify the PR Analytics page loads correctly.

Screenshots or videos of changes:

Before
Screenshot 2025-10-08 at 8 42 31 PM
After
Screenshot 2025-10-08 at 5 41 29 PM

Note:

  • No backend changes were required for this fix.
  • Tested locally for admin users.
  • Only need to test that the route is working; no need to test for dark mode.

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 8, 2025

Deploy Preview for highestgoodnetwork-dev failed.

Name Link
🔨 Latest commit a5dde96
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6940a3d70541fb00083ed22a

@somramnani somramnani added the do not review Do not review or look at code without full context label Oct 8, 2025
@somramnani somramnani force-pushed the som_ramnani_fixes_broken_pr_analytic_link branch from 88a0b9f to 6824e01 Compare October 8, 2025 22:31
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 8, 2025

@somramnani somramnani added Medium Priority and removed do not review Do not review or look at code without full context labels Oct 9, 2025
@hemanthvenkat
Copy link
Copy Markdown

Verified the PR Analytics page — it loads correctly when accessed via Other Links → PR Analytics or directly through /pr-dashboard/analytics.
All components render properly, and the page functions as expected for the Admin user.
https://github.com/user-attachments/assets/31d482ba-4d1c-44ae-9193-e568d3bba87e

@sonarqubecloud
Copy link
Copy Markdown

@somramnani somramnani changed the title Som fixes PR Analytic Link som-fix/pr-analytic-link Dec 16, 2025
@somramnani somramnani changed the title som-fix/pr-analytic-link (som)fix/pr-analytic-link Dec 16, 2025
@somramnani somramnani changed the title (som)fix/pr-analytic-link som-fix/pr-analytic-link Dec 16, 2025
@somramnani somramnani changed the title som-fix/pr-analytic-link som-fix/broken-pr-analytic-link Dec 16, 2025
@somramnani somramnani closed this Dec 16, 2025
@somramnani somramnani deleted the som_ramnani_fixes_broken_pr_analytic_link branch December 16, 2025 00:20
@somramnani somramnani restored the som_ramnani_fixes_broken_pr_analytic_link branch December 16, 2025 00:23
@somramnani somramnani deleted the som_ramnani_fixes_broken_pr_analytic_link branch December 16, 2025 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants