Skip to content

feat(API): Add q search param to locales list endpoint#1127

Merged
Hassan Ahmed (hahmed-dev) merged 2 commits into
mainfrom
feat/SCM-1433-add-q-param-to-locales-list
May 15, 2026
Merged

feat(API): Add q search param to locales list endpoint#1127
Hassan Ahmed (hahmed-dev) merged 2 commits into
mainfrom
feat/SCM-1433-add-q-param-to-locales-list

Conversation

@hahmed-dev
Copy link
Copy Markdown
Contributor

Summary

Adds optional q query parameter to the (List Locales) endpoint
Enables case-insensitive prefix filtering of tags by name (e.g. ?q=name:en)
Parameter type: string, optional — existing callers are unaffected

@hahmed-dev
Copy link
Copy Markdown
Contributor Author

Screenshot 2026-05-13 at 10 43 30

@hahmed-dev Hassan Ahmed (hahmed-dev) force-pushed the feat/SCM-1433-add-q-param-to-locales-list branch 2 times, most recently from 7b551bc to 29083fd Compare May 13, 2026 08:55
- Add q parameter to localesList API call in LocalesApiTest
- Align test signature with updated locales list endpoint
@hahmed-dev Hassan Ahmed (hahmed-dev) force-pushed the feat/SCM-1433-add-q-param-to-locales-list branch from 29083fd to 78d0a53 Compare May 13, 2026 09:03
@hahmed-dev Hassan Ahmed (hahmed-dev) requested a review from a team May 13, 2026 09:07
Copy link
Copy Markdown
Collaborator

@jablan jablan left a comment

Choose a reason for hiding this comment

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

some more clarification of the q parameter

Comment thread paths/locales/index.yaml Outdated
Address PR review feedback — describe the q parameter's argument syntax
and prefix-match semantics rather than just "filter by name".

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@jablan jablan changed the title feat(locales): Add q search param to locales list endpoint feat(API): Add q search param to locales list endpoint May 15, 2026
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.

2 participants