Skip to content

Conversation

bidoubiwa
Copy link
Contributor

Pull Request

What does this PR do?

Previously instant-meilisearch added crop markers in the specific fields depending on the snippetEllipsisText value provided by the configuration widget of instantsearch. The injection of the markers was done by instant-meilisearch. Now it is added as the query parameter cropMarker to the search request where the markers are added by meilisearch the engine.

Some tests have been removed as they where not relevant.

Base automatically changed from fix_tests to bump-meilisearch-v0.27.0 April 21, 2022 12:13
Copy link
Contributor

@alallema alallema 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 bidoubiwa merged commit d506846 into bump-meilisearch-v0.27.0 Apr 21, 2022
@bidoubiwa bidoubiwa deleted the update_snippeting_resolver branch April 21, 2022 14:03
@bidoubiwa bidoubiwa added the skip-changelog The PR will not appear in the release changelogs label Apr 21, 2022
@bidoubiwa bidoubiwa added enhancement New feature or request and removed skip-changelog The PR will not appear in the release changelogs labels May 5, 2022
meili-bors bot added a commit that referenced this pull request May 9, 2022
749: Update version for the next release (v0.7.1) r=brunoocasali a=bidoubiwa

This version makes this package compatible with Meilisearch v0.27.0 🎉
Check out the changelog of [Meilisearch v0.27.0](https://github.com/meilisearch/meilisearch/releases/tag/v0.27.0) for more information on the changes.

## 🚀 Enhancements

- Add highlight prefix suffix tags #739
- Add crop marker support #738

Thanks again to `@bidoubiwa` and `@mmachatschek!` 🎉


Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com>
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.

2 participants