-
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 facetstats in the searchResults #443
Add facetstats in the searchResults #443
Conversation
442: Failing tests due to breaking patch in mockito r=irevoire a=bidoubiwa Mockito released a [new patch version](https://github.com/lipanski/mockito/releases/tag/0.32.5) messing with the asynchronousity methods. Due to that, the tests were panicking in the other PR's. Since additionally to that I received a lot of warning from the library saying that the methods we were using are deprecated, I updated mockito to its v1 version. Co-authored-by: Charlotte Vermandel <[email protected]>
…ump-meilisearch-v1.1.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.
Well done! 🥖
outer_fn | ||
.attrs | ||
.push(parse_quote!(#[futures_await_test::async_test])); | ||
outer_fn.attrs.push(parse_quote!(#[tokio::test])); |
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.
Oh, what is tokio test?
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.
Its the library used to handle async testing. We had to change it to be compatible with the new version of Mockito
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! 🎉
456: Update version for the next release (v0.23.0) r=bidoubiwa a=meili-bot This version makes this package compatible with Meilisearch v1.1 🎉 Check out the changelog of [Meilisearch v1.1](https://github.com/meilisearch/meilisearch/releases/tag/v1.1.0) for more information on the changes. ## 🚀 Enhancements * Add the `multi_search` method on `Client` #454 ##⚠️ Breaking changes * Rename `Document` trait to `IndexConfig` and extend with function to retrieve index (#445) `@amaihoefner` * Add facetstats in the searchResults #443 * Add a new `csv_delimiter` parameter on `index.add_or_replace_unchecked_payload` and `index. add_or_update_unchecked_payload` #455 Thanks again to `@amaihoefner,` and `@shimatar0!` 🎉 Co-authored-by: meili-bot <[email protected]>
* Failing tests due to breaking patch in mockito * Add facetstats for Meilisearch v1.1.0 * Update src/tasks.rs --------- Co-authored-by: bors[bot] <26634292+bors[bot]@users.noreply.github.com>
Add the
facetStats
type in theSearchResults
structure. It introduces min max values for numeric facet that have been required through the search parameter facetAs per the specification
SDK requirements: meilisearch/integration-guides#251