Skip to content

BYNT-807: Add search to location admin levels and types#112

Merged
tarekio merged 2 commits intomainfrom
bynt-807-loc-type-lal-search
Jun 18, 2025
Merged

BYNT-807: Add search to location admin levels and types#112
tarekio merged 2 commits intomainfrom
bynt-807-loc-type-lal-search

Conversation

@cango91
Copy link
Contributor

@cango91 cango91 commented Jun 17, 2025

Jira Issue

  1. BYNT-807

Description

GET/POST /admin/api/location-admin-levels and /admin/api/location-types now support search by title as query parameter.

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

 ## Jira Issue
1. [BYNT-807](https://syriajustice.atlassian.net/browse/BYNT-807)

 ## Description
 `GET/POST` /admin/api/location-admin-levels and /admin/api/location-types now support search by title as query parameter.

 ## Checklist
- [ ] Tests added/updated
- [ ] Documentation updated (if needed)
- [ ] New strings prepared for translations

 ## API Changes (if applicable)
- [ ] Permissions checked
- [ ] Endpoint tests added

 ## Additional Notes
[Any other relevant information]
@cango91 cango91 requested a review from tarekio June 17, 2025 03:06
@tarekio tarekio requested a review from level09 June 17, 2025 18:13
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

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

The is fine / Approved, since those table won't have massive row count, so we can get away without adding indexes.

@tarekio tarekio merged commit de8d39c into main Jun 18, 2025
6 of 7 checks passed
@tarekio tarekio deleted the bynt-807-loc-type-lal-search branch June 18, 2025 18:45
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.

3 participants