Skip to content

Neeraj Fix Multi Category Filter#4470

Merged
one-community merged 8 commits intodevelopmentfrom
Neeraj_Fix_Multi_Category_Filter_Frontend
Dec 17, 2025
Merged

Neeraj Fix Multi Category Filter#4470
one-community merged 8 commits intodevelopmentfrom
Neeraj_Fix_Multi_Category_Filter_Frontend

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Nov 29, 2025

Description

image

Related PRS (if any):

This frontend PR is related to the OneCommunityGlobal/HGNRest#1932 backend PR.

Main changes explained:

  • src/components/Collaboration/Collaboration.jsx
    • Updated fetchJobAds() to properly join multiple categories into a comma-separated string.
    • Ensured toggleCategory() triggers fetch using the updated category list.
  • src/components/Collaboration/Collaboration.module.css
    • Fixed job card container layout to prevent cards from stacking incorrectly.
  • Added folder for upcoming UI tests to validate category filtering functionality.

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. Navigate to : http://localhost:5173/Collaboration
  6. Select multiple categories (e.g., Software & IT, Marketing, Engineering) and verify multiple categories can be selected.

Screenshots or videos of changes:

image image

Note:

Include the information the reviewers need to know.

@Neeraj-Kondaveeti Neeraj-Kondaveeti added the do not review Do not review or look at code without full context label Nov 29, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 29, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit b830d8e
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69420736c8729600080d0730
😎 Deploy Preview https://deploy-preview-4470--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.

@Neeraj-Kondaveeti Neeraj-Kondaveeti changed the title Fix multi-category filter UI/UX and update Header.jsx to resolve runt… Neeraj_Fix_Multi_Category_Filter Dec 1, 2025
@Neeraj-Kondaveeti Neeraj-Kondaveeti changed the title Neeraj_Fix_Multi_Category_Filter Neeraj Fix Multi Category Filter Dec 2, 2025
@Neeraj-Kondaveeti Neeraj-Kondaveeti added Medium Priority and removed do not review Do not review or look at code without full context labels Dec 2, 2025
Anusha-Gali
Anusha-Gali previously approved these changes Dec 3, 2025
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 Neeraj,

I have reviewed your PR locally and am able to filter multiple jobs.
Screenshot 2025-12-02 at 11 36 41 PM
Screenshot 2025-12-02 at 11 34 12 PM
Screenshot 2025-12-02 at 11 34 19 PM
Screenshot 2025-12-02 at 11 34 27 PM
Screenshot 2025-12-02 at 11 34 37 PM

Prem203
Prem203 previously approved these changes Dec 6, 2025
Copy link
Copy Markdown
Contributor

@Prem203 Prem203 left a comment

Choose a reason for hiding this comment

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

Reviewed the PR with an admin account and the multiple job categories can now be selected from the dropdown menu.

image

Copy link
Copy Markdown

@IndraAnuraag9 IndraAnuraag9 left a comment

Choose a reason for hiding this comment

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

I have reviewed this PR locally using an admin account. I am able to select multiple categories and retrieve results successfully.
Screenshot 2025-12-05 at 11 10 26 PM
Screenshot 2025-12-05 at 11 10 42 PM
Screenshot 2025-12-05 at 11 14 50 PM

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit d3eed17 into development Dec 17, 2025
10 checks passed
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.

5 participants