meilisearch / meilisearch-go

Golang wrapper for the Meilisearch API

https://www.meilisearch.com

MIT License

519 stars 85 forks source link

Add facet search method #543

Closed migueltarga closed 2 months ago

migueltarga commented 2 months ago

Pull Request

Related issue

Fixes #470

What does this PR do?

Implements Facet Search introduced on version 1.3 Reference: https://www.meilisearch.com/docs/reference/api/facet_search

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!

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.77%. Comparing base (aea59d3) to head (c4e85d6).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #543 +/- ## ========================================== + Coverage 86.51% 86.77% +0.25% ========================================== Files 10 11 +1 Lines 1602 1633 +31 ========================================== + Hits 1386 1417 +31 Misses 117 117 Partials 99 99 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

meili-bors[bot] commented 2 months ago

Build succeeded:

migueltarga commented 2 months ago

@curquiza Thank you for the approval!

curquiza commented 2 months ago

I wait for a few days to see if I can merge https://github.com/meilisearch/meilisearch-go/pull/544 as well and put it in the same release If I have no answer I will release on Wednesday