Bosu - fix: enable HGN questionnaire profile navigation - #5462
Open
BosuBose132 wants to merge 1 commit into
Open
Bosu - fix: enable HGN questionnaire profile navigation#5462BosuBose132 wants to merge 1 commit into
BosuBose132 wants to merge 1 commit into
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
BosuBose132
changed the title
iAbhi001
self-requested a review
August 23, 2026 03:14
iAbhi001
approved these changes
Aug 23, 2026
iAbhi001
left a comment
iAbhi001
left a comment
There was a problem hiding this comment.
PR Review & Validation Summary
Tested locally alongside backend PR #2315. Everything works as expected.
Verification Checklist:
- Navigation: Successfully navigated from
/hgnhelp/communityto/hgnhelp/profile/<userId>by clicking user cards. - Routing & ID handling: Verified that the actual
userIdis used instead of the questionnaire_id, and fallback handling for missing IDs works cleanly. - API & Data Rendering: Confirmed
/api/skills/profile/<userId>is called correctly and populates user name, skills, experience, and team info (no "Unknown User" fallback). - Edge Cases: Tested users with and without team assignments, verified direct page refresh on the profile URL, and checked dark mode styling.
- Tests & Build: Local test suites and production build executed without errors.
LGTM! Approved. 🚀
linlin-husky
approved these changes
Aug 29, 2026
linlin-husky
left a comment
linlin-husky
left a comment
There was a problem hiding this comment.
Hi, BosuBose132,
PR Review & Validation Summary
I have reviewed and tested this PR locally alongside the backend branch Bosu-Fix-HGN-Questionnaire-Profile-ID (PR #2315). Everything works as expected!
Functional Verification Checklist:
- Navigation & Routing: Successfully navigated to
/hgnhelp/profile/<userId>by clicking user cards from the community list. Verified that the actualuserIdis used instead of the questionnaire response_id. - API & Data Rendering: Confirmed the frontend fetches
/api/skills/profile/<userId>with HTTP 200 and correctly populates the user's name, skills radar chart, leadership/additional experience, and team info (no "Unknown User" placeholders). - Multi-User & Edge Cases: Tested multiple profiles (both with and without assigned teams). Direct browser refresh on the profile URL loads the profile seamlessly.
- UI & Theming: Verified that dark mode styling and radar chart elements render cleanly.
Test & Build Verification:
- Unit Tests: All test suites passed (
324 passed / 2171 tests passed). - Lint & Formatting: ESLint and
git diff --checkcompleted without errors. - Build: Production build (
npm run build) succeeded without warnings/errors.
Approved.
DeepighaJ
requested changes
Aug 29, 2026
DeepighaJ
left a comment
DeepighaJ
left a comment
There was a problem hiding this comment.
Hi @BosuBose132
Checked out to frontend branch along side development as backend is already merged.
I navigated to /hgnhelp/community but on clicking on any user profile I don't see redirection to /hgnhelp/profile/
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes the HGN Questionnaire Dashboard profile route accessibility issue.
The route
/hgnhelp/profile/:userIdexisted, but users displayed in the HGN community list did not have a working navigation path to their questionnaire profile. During investigation, it was also identified that the profile page needed to use the HGN skills profile endpoint to retrieve the user's questionnaire skills, experience information, and team details.The frontend now provides profile navigation from community user cards using the actual user profile ID supplied by the backend and loads the combined HGN skills profile data.
Related PRs
UserProfileID asuserIdin ranked and community questionnaire responses.Main changes explained
UserCard.jsxto make the user's name link to/hgnhelp/profile/:userId.user.userIdinstead of the HGN questionnaire response document_id.userIddo not generate an invalid profile URL.UserProfilePage.jsxto useENDPOINTS.SKILLS_PROFILE(userId)so the page receives questionnaire skills, experience/follow-up information, and team data.How to test
Checkout this frontend branch.
Checkout the related backend PR branch.
Run the backend locally.
Run the frontend with:
npm run start:localLog in with an authorized test account.
Navigate to
/hgnhelp/community, or access the community list through the HGN questionnaire flow.Click a user's name.
Verify the browser navigates to:
/hgnhelp/profile/<userId>Verify the profile displays the selected user's:
Open DevTools → Network and verify the profile request uses:
/api/skills/profile/<userId>Verify the endpoint returns the selected user's real profile instead of the
Unknown Userplaceholder.Test multiple users, including a user with team information and a user without team information.
Refresh the profile route directly and confirm it continues to load correctly.
Verify the profile displays correctly in dark mode.
Validation performed
git diff --check: passednpm run build: passedScreenshots or videos of changes
Uploading Screen Recording 2026-08-22 at 11.52.29 AM.mov…
Note
The duplicate existing
/hgnhelproute definitions were investigated but were not modified because they are not required for this profile-route fix. No unrelated routing, dependency, or backend changes are included in this frontend PR.