Siri Sudheeksha Vavila: Fix hgn questionnaire skill filter user details#5205
Conversation
…ll to CommunityMembersPage
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
- Pass darkMode prop to SkillFilterButtons and PreferenceFilterButtons so filter buttons render with dark styling in dark mode - Add dark border and dark background for accordion border and content area in CommunityMembersPage.module.css - Remove hardcoded white background from .userWrapper in RankedUserList.module.css so dark mode background shows through correctly
…tion and remove ambiguous img spacing
fixed the changes and requested re- review
fixed the "Top 15 Community Members" page UI and the layout and styling
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Sudheeksha,
I tested this PR locally. Functionality works as expected and the dark mode alignment on the Community Members page looks good. One remaining issue though, the table borders in the "Top 15 Community Members" section are not properly aligned with the table structure in dark mode. Please fix the border alignment so they line up correctly with the table rows and columns.
…s, remove duplicate CSS overrides
|
|
Thank you all, merging! |













Description
Fixes the TopCommunityMembers component as it was not displaying any member data after selecting a skill from the dropdown, and navigating to the Community Members page via "Show your team members" required manually re-selecting filters before any members would appear.
Root causes:
TopCommunityMembers was calling /userProfile/skills/:skill and /team-skills/:skill — endpoints that return empty arrays for most users because they depend on HGN form responses being linked to the logged-in user's team. Members were never shown.
The "Show your team members" link navigated to /hgnhelp/community with no state, so CommunityMembersPage started with no filters selected, hiding the member list entirely.
Related PRS (if any):
This frontend PR is related to PR 4214
…
Main changes explained:
…
How to test:
Screenshots or videos of changes:
Before:
After :
Note:
Include the information the reviewers need to know.