Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix/BE search with tags #493

Merged
merged 3 commits into from
Dec 15, 2024
Merged

Fix/BE search with tags #493

merged 3 commits into from
Dec 15, 2024

Conversation

oguzpancuk
Copy link
Contributor

Implemented post search with multiple tags. Listed on swagger as posts/posts-by-tags

A new endpoint for searching posts with multiple tags is implemented
@oguzpancuk oguzpancuk added bug Something isn't working priority: high Very urgent issue workload: 2 issues that are requiring medium effort backend labels Dec 14, 2024
@oguzpancuk oguzpancuk added this to the 451 - Milestone 3 milestone Dec 14, 2024
@oguzpancuk oguzpancuk self-assigned this Dec 14, 2024
@oguzpancuk oguzpancuk linked an issue Dec 14, 2024 that may be closed by this pull request
2 tasks
@oguzpancuk oguzpancuk changed the title Fix/be search with tags Fix/BE search with tags Dec 15, 2024
Get method now takes one additional parameter ("and" or default "or" behaviour) and performs the query accordingly.
Copy link
Contributor

@rukiyeaslan rukiyeaslan left a comment

Choose a reason for hiding this comment

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

nice work! lgtm

@rukiyeaslan
Copy link
Contributor

@oguzpancuk I cannot merge this pr due to a conflict, could you resolve it?

@oguzpancuk oguzpancuk merged commit 50a18d3 into dev Dec 15, 2024
@oguzpancuk
Copy link
Contributor Author

oguzpancuk commented Dec 15, 2024

@oguzpancuk I cannot merge this pr due to a conflict, could you resolve it?

I resolved the conflict and merged the branch. Thank you for your review, @rukiyeaslan .

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backend bug Something isn't working priority: high Very urgent issue workload: 2 issues that are requiring medium effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] Implement Post Search With Multiple Tags
2 participants