Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Return facets even when there is no value associated to it #518

Merged
merged 2 commits into from
Apr 28, 2022

Conversation

Kerollmops
Copy link
Member

This PR is related to meilisearch/meilisearch#2352 and should fix the issue when Meilisearch is up-to-date with this PR.

@Kerollmops Kerollmops added the no breaking The related changes are not breaking (DB nor API) label Apr 26, 2022
Copy link
Member

@irevoire irevoire left a comment

Choose a reason for hiding this comment

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

👍

@Kerollmops
Copy link
Member Author

Kerollmops commented Apr 27, 2022

I will let @curquiza decide on whether we merge it now/here or not.

@curquiza
Copy link
Member

curquiza commented Apr 27, 2022

What would be the issue to merge it @Kerollmops?
If it's technically ok for you, feel free to merge it into main. I will release milli v0.26.4 (on another branch than main)

@Kerollmops
Copy link
Member Author

Thank you, I merge this!
bors merge

@bors
Copy link
Contributor

bors bot commented Apr 28, 2022

@bors bors bot merged commit 9db86aa into main Apr 28, 2022
@bors bors bot deleted the return-empty-facet-distrib branch April 28, 2022 10:05
bors bot added a commit that referenced this pull request May 3, 2022
519: Release v0.26.4: returns facets even when there is no value associated to it + bug fix during indexation r=curquiza a=curquiza

I cherry-picked the commits from these PRs
- #518
- #522

`release-v0.26.4` has been started from the tag `v0.26.3` and not from main

The release tag `v0.26.4` will be done on the branch `release-v0.26.4` once this PR is merge

Co-authored-by: Kerollmops <clement@meilisearch.com>
Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
bors bot added a commit that referenced this pull request May 3, 2022
519: Release v0.26.4: returns facets even when there is no value associated to it + bug fix during indexation + bug fix on typo-tolerance r=Kerollmops a=curquiza

I cherry-picked the commits from these PRs
- #518
- #522
- #520

`release-v0.26.4` has been started from the tag `v0.26.3` and not from main

The release tag `v0.26.4` will be done on the branch `release-v0.26.4` once this PR is merge

Co-authored-by: Kerollmops <clement@meilisearch.com>
Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
Co-authored-by: ad hoc <postma.marin@protonmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
no breaking The related changes are not breaking (DB nor API)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants