Skip to content

Conversation

@daniel-shuy
Copy link
Contributor

Fixes #369

Add support for * in attributesToSnippet to snippet all attributes, as documented in https://www.algolia.com/doc/api-reference/api-parameters/attributesToSnippet/#usage-notes

@daniel-shuy daniel-shuy force-pushed the attributesToSnippet-support-asterisk branch from 680fec3 to 86eee40 Compare October 12, 2021 14:43
@daniel-shuy daniel-shuy force-pushed the attributesToSnippet-support-asterisk branch from 0a42c16 to f20a7d0 Compare October 14, 2021 14:35
Copy link
Contributor

@bidoubiwa bidoubiwa left a comment

Choose a reason for hiding this comment

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

LGTM 🔥🔥🔥

@bidoubiwa
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 18, 2021

Build succeeded:

@bors bors bot merged commit 0cec448 into meilisearch:main Oct 18, 2021
@bidoubiwa
Copy link
Contributor

If you are participating in Hacktoberfest, and you would like to receive a small gift from MeiliSearch too, please complete this form.

@daniel-shuy daniel-shuy deleted the attributesToSnippet-support-asterisk branch October 18, 2021 20:59
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.

empty _snippetResult when configuring attributesToSnippet

2 participants