-
Notifications
You must be signed in to change notification settings - Fork 90
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.
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
Add feature: dictionary settings #529
Add feature: dictionary settings #529
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @hasan-raja
thanks a lot for your PR
looks the Ci rust-format is failing, otherwise the PR is good
can you fix it before I merge?
Hi @curquiza made an update based on rust format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
bors merge
Build succeeded: |
Pull Request
Related issue
Fixes #527
What does this PR do?
dictionary
setting to theSettings
struct, providing users with a flexible way to manage their search dictionary.dictionary
setting:get_dictionary
: Retrieves the current dictionary configuration.set_dictionary
: Updates the dictionary with a new configuration.reset_dictionary
: Resets the dictionary configuration to default values.PR checklist
Please check if your PR fulfills the following requirements:
.code-samples.meilisearch.yaml
file to incorporate new examples for theget_dictionary
,set_dictionary
, andreset_dictionary
methods.Please review and provide feedback on the changes introduced in this pull request.