Skip to content

feat: filter collection based on name #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

feat: filter collection based on name #1

wants to merge 11 commits into from

Conversation

sanzog03
Copy link
Collaborator

@sanzog03 sanzog03 commented May 1, 2025

What I am changing

  • Add the ability to filter collections based on collection ID.

How I did it

  • Add a query parameter depdency injection which checks for collection ID substring regex, validates the regex and uses that to filter the CollectionList.

How you can test it

  • Using the API docs, check the list of collections without any filter. Then, select any collection and use part of the collection ID as input to the collectionID regex filter to check the result.
  • A test is added to the code.

Related Issues

@sanzog03 sanzog03 changed the title Task/filter collection based on name feat: filter collection based on name May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slesaad slesaad Awaiting requested review from slesaad

@paridhi-parajuli paridhi-parajuli Awaiting requested review from paridhi-parajuli

@amarouane-ABDELHAK amarouane-ABDELHAK Awaiting requested review from amarouane-ABDELHAK

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sanzog03 @vincentsarago