Skip to content

Conversation

@ellnix
Copy link
Contributor

@ellnix ellnix commented Jan 1, 2025

Currently, the description of the locales search param does not make sense. When I read it it felt like a word was missing. Apologies if this was intended and correct.

In addition changed "language" to "language(s)" since more than one language may be specified.

Currently, the description of the `locales` search param does not make
sense. When I read it it felt like a word was missing. Apologies if this
was intended and correct.

In addition changed "language" to "language(s)" since more than one
language may be specified.
guimachiavelli
guimachiavelli previously approved these changes Jan 14, 2025
Copy link
Member

@guimachiavelli guimachiavelli left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! I removed the parentheses as they're not particularly necessary, but looks good to me!

@guimachiavelli guimachiavelli merged commit 5295f32 into meilisearch:main Jan 14, 2025
1 check 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.

2 participants