Skip to content

Conversation

@curquiza
Copy link
Member

@curquiza curquiza commented Oct 4, 2021

@curquiza
Copy link
Member Author

curquiza commented Oct 4, 2021

@bidoubiwa, docs-scraper does not currenlty work with MeiliSearch v0.22.0 and later, so we should merge it and release it asap :)

@curquiza curquiza requested a review from bidoubiwa October 4, 2021 17:26
@curquiza
Copy link
Member Author

curquiza commented Oct 4, 2021

I think this is breaking for the user since we enforce the usage of v0.22.0 of MeiliSearch and later

@curquiza curquiza changed the title Fix to make it work for MeiliSearch v0.22.0 and later Fix asc/dsc syntax: make it work for MeiliSearch v0.22.0 and later Oct 4, 2021
@curquiza curquiza added the breaking-change The related changes are breaking for the users label Oct 4, 2021
@curquiza curquiza changed the title Fix asc/dsc syntax: make it work for MeiliSearch v0.22.0 and later Fix asc/desc syntax: make it work for MeiliSearch v0.22.0 and later Oct 4, 2021
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.

My bad for not fixing this during the previous release. LGTM !

@bidoubiwa
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Oct 5, 2021
158: Fix asc/desc syntax: make it work for MeiliSearch v0.22.0 and later r=bidoubiwa a=curquiza

Related to https://github.com/meilisearch/MeiliSearch/discussions/1766

Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
@bidoubiwa bidoubiwa merged commit 4e1c027 into main Oct 5, 2021
@bidoubiwa bidoubiwa deleted the hotfix branch October 5, 2021 14:15
@curquiza
Copy link
Member Author

curquiza commented Oct 5, 2021

I totally missed it as well, don't worry! :)

@bors
Copy link
Contributor

bors bot commented Oct 5, 2021

Timed out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change The related changes are breaking for the users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants