Sai Teja: Fix PR Analytics broken page – Add contextual error handling#4838
Sai Teja: Fix PR Analytics broken page – Add contextual error handling#4838one-community merged 18 commits intodevelopmentfrom
Conversation
Frontend Release to Main [4.62]
Frontend Release to Main [4.63]
Just the 4497 fix to Main
Frontend Release to Main [4.64]
Frontend Release to Main [4.65]
Frontend Release to Main [4.66]
Frontend Release to Main [4.67]
Frontend Release to Main [4.68]
Frontend Release to Main [4.69]
Frontend Release to Main [4.70]
Frontend Release to Main [4.71]
Frontend Release to Main [4.72]
Frontend Release to Main [4.73]
Frontend Release to Main [4.74]
Frontend Release to Main [4.75]
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
saitejakaasoju
changed the title
one-community
added
High Priority - Please Review First
HemanthNidamanuru
left a comment
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Sai Teja,
I have reviewed your PR locally. However, I was unable to validate the contextual message "PR Analytics is temporarily unavailable." as random data is loading at that address instead of showing the expected fallback message.
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Sai Teja,
I have reviewed your PR locally and the route works as per requirement including the contextual error handling.
…tics-friendly-error # Conflicts: # src/components/NotFound/NotFoundPage.jsx
|
|
Thank you all, merging! |
rajanidi1999
left a comment
rajanidi1999
left a comment
There was a problem hiding this comment.
Hi Sai,
I have reviewed your PR as per given steps and its successfully executed.
6 participants
Description
Related PRS (if any):
This frontend PR is not related to any backend PR.
Main changes explained:
/pr-dashboardpathsHow to test:
check into current branch
do
npm installand...to run this PR locallyClear site data/cache
Login as Admin User
Simulate PR Analytics failing to load:
Temporarily comment out the PR Dashboard route in src/routes.jsx:
Navigate to:
Other Links → PR Team Analytics (or directly open /pr-dashboard/overview)
Verify:
You see the contextual message: “PR Analytics is temporarily unavailable.”
The “Go to Reports Dashboard” link works.
Verify a random invalid route (e.g., /random123) still shows the normal rabbit “Page Not Found”.
Screenshots or videos of changes:
Previous:
Now:
Note: