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

Add methods to configure new prefixSearch and facetSearch index settings #702

Merged
merged 6 commits into from
Dec 6, 2024

Conversation

Strift
Copy link
Collaborator

@Strift Strift commented Dec 5, 2024

Pull Request

Related issue

Fixes #694

What does this PR do?

  • Add tests to make sure getSettings(), and updateSettings allow the new settings
  • Add the getFacetSearch(), updateFacetSearch(), and resetFacetSearch() methods
  • Add the getPrefixSearch(), updatePrefixSearch(), and resetPrefixSearch() methods
  • Add tests

@Strift Strift linked an issue Dec 5, 2024 that may be closed by this pull request
4 tasks
@Strift Strift requested a review from norkunas December 5, 2024 09:52
@Strift
Copy link
Collaborator Author

Strift commented Dec 5, 2024

Hey @norkunas 👋 I went ahead I tagged you as a reviewer since you helped me with the previous PRs. I hope you can assist me with reviewing this one too. 🙏

// Settings - Facet Search

/**
* @since Meilisearch v1.12.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we really need this annotation (everywhere)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I wouldn't say we need it to the point where we should update all the code base. But if it shows up in the generated documentation, then it's useful information for the reader. What do you think?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok

@Strift Strift merged commit 3e0a026 into bump-meilisearch-v0.12.0 Dec 6, 2024
30 checks passed
@Strift Strift deleted the feat/add-v1.12-settings branch December 6, 2024 07:43
@Strift Strift changed the title Add prefixSearch and facetSearch index settings Add methods to configure new prefixSearch and facetSearch index settings Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v1.12.0] New settings: facetSearch & prefixSearch
2 participants