-
Notifications
You must be signed in to change notification settings - Fork 115
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
Changes related to the next Meilisearch release (v0.28.0) #403
Conversation
Change HTTP methods
Apply search changes
Co-authored-by: Bruno Casali <[email protected]>
Add URLBuilder for query parameters
Apply tasks changes
Rename Result class to Results
Apply dumps changes
Co-authored-by: cvermand <[email protected]>
Update Index API for Meilisearch v.28
Update Keys API for Meilisearch v.28
Co-authored-by: cvermand <[email protected]>
Update documents API for Meilisearch v.28
Update Tenant Token API for Meilisearch v.28
Refactor handlers for the usage of URLBuilder
Co-authored-by: Bruno Casali <[email protected]>
Co-authored-by: Bruno Casali <[email protected]>
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.
Perfect perfect! 🌮
Code samples changes related to Meilisearch release (v0.28.0)
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.
LGTM 🔥
protected int estimatedTotalHits; | ||
protected Object facetDistribution; |
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.
Why are you removing nbHits
here and making the changes on facetDistribution?
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.
What do you mean?
meilisearch/meilisearch#2376
Related to this issue: meilisearch/integration-guides#205
This PR:
This PR is auto-generated for the pre-release week purpose.
Done: