Skip to content

Neeraj Fix Multi Category Filtering#1932

Merged
one-community merged 1 commit intodevelopmentfrom
Neeraj_Fix_Multi_Category_Filter_Backend
Dec 17, 2025
Merged

Neeraj Fix Multi Category Filtering#1932
one-community merged 1 commit intodevelopmentfrom
Neeraj_Fix_Multi_Category_Filter_Backend

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Dec 2, 2025

Description

image

Related PRS (if any):

To test this backend PR you need to checkout the OneCommunityGlobal/HighestGoodNetworkApp#4470 frontend PR.

Main changes explained:

  • Updated jobController.js to correctly support multi category filtering.
  • Simplified backend query conditions.

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. Confirm API responses using postman :

Screenshots or videos of changes:

image

Note:

  • This PR only modifies backend filtering logic.
  • No scheme or data changes were made.
  • All backend coverage tests were run and passed.
Screenshot 2025-12-02 at 5 23 47 PM
  • Frontend PR depends on the backend PR for correct results.

@Neeraj-Kondaveeti Neeraj-Kondaveeti added the Do Not Review Do not review or look at code without full context label Dec 2, 2025
@Neeraj-Kondaveeti Neeraj-Kondaveeti removed the Do Not Review Do not review or look at code without full context label Dec 2, 2025
@Neeraj-Kondaveeti Neeraj-Kondaveeti changed the title Neeraj Fix: Multi Category Filtering Neeraj Fix Multi Category Filtering Dec 2, 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 and the API works as per requirement.
Screenshot 2025-12-02 at 11 37 04 PM
Screenshot 2025-12-02 at 11 37 15 PM
Screenshot 2025-12-02 at 11 37 56 PM
Screenshot 2025-12-02 at 11 38 16 PM

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. The API is returning apt results when multiple categories are selected.
Screenshot 2025-12-05 at 10 57 51 PM
Screenshot 2025-12-05 at 11 00 12 PM
Screenshot 2025-12-05 at 11 14 42 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 8f2cc56 into development Dec 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants