Skip to content
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

Support search in facet values #839

Merged
merged 5 commits into from
Sep 4, 2023
Merged

Conversation

cyprx
Copy link
Contributor

@cyprx cyprx commented Sep 3, 2023

Pull Request

Related issue

Fixes #830

What does this PR do?

  • Add function facet_search which performs search in facet values
  • Add code samples
  • Add integration tests

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@cyprx cyprx marked this pull request as ready for review September 3, 2023 09:22
@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.07% 🎉

Comparison is base (81e61ea) 92.13% compared to head (565bfd2) 92.20%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #839      +/-   ##
==========================================
+ Coverage   92.13%   92.20%   +0.07%     
==========================================
  Files          13       13              
  Lines         776      783       +7     
==========================================
+ Hits          715      722       +7     
  Misses         61       61              
Files Changed Coverage Δ
meilisearch/config.py 100.00% <100.00%> (ø)
meilisearch/index.py 99.60% <100.00%> (+<0.01%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@sanders41 sanders41 left a comment

Choose a reason for hiding this comment

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

Thanks @cyprx, just a couple minor update requests.

tests/conftest.py Outdated Show resolved Hide resolved
meilisearch/index.py Show resolved Hide resolved
@sanders41 sanders41 added the enhancement New feature or request label Sep 4, 2023
Copy link
Collaborator

@sanders41 sanders41 left a comment

Choose a reason for hiding this comment

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

LGTM! @cyprx the test failures have nothing to do with your code. Codecov seems to be down right now. I'll re-run shortly and it should resolve.

@sanders41 sanders41 mentioned this pull request Sep 4, 2023
3 tasks
@sanders41
Copy link
Collaborator

bors merge

@meili-bors
Copy link
Contributor

meili-bors bot commented Sep 4, 2023

@meili-bors meili-bors bot merged commit 7f4146d into meilisearch:main Sep 4, 2023
10 checks passed
meili-bors bot added a commit that referenced this pull request Sep 11, 2023
841: Bump github actions checkout to v4 r=curquiza a=sanders41

# Pull Request

GitHub Actions are currently failing across multiple repositories with network issues. the checkout action is currently having issues with this, issue [here](actions/checkout#1448), and bumping to the newly released v4 seems to be helping. This action is not the only thing that is having issues with the network problems so we could still see failures until it is resolved, but bumping to v4 is at least helping with the checkout part.

These network issues are also what keep causing #839 to fail.

## Related issue
Fixes #<issue_number>

## What does this PR do?
- Bumps checkout to v4

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Paul Sanders <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v1.3] Search in facet values facetSearch
3 participants