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

Make use of Meilisearch sorting #26

Closed
wants to merge 2 commits into from

Conversation

robbanl
Copy link
Contributor

@robbanl robbanl commented Oct 31, 2023

Make use of Statamic score sorting so we can have the results sorted as they appear in Meilisearch and not by UUIID in Statamic.

Copy link
Member

@duncanmcclean duncanmcclean left a comment

Choose a reason for hiding this comment

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

Thanks for this! 🎉

I've had to make a new PR (#27) for your changes since I wanted to make a couple of tweaks to better match how it's done in the core Algolia driver.

For any future PRs, please don't commit onto main/master on your fork as GitHub doesn't allow maintainers to push to those branches.

@robbanl
Copy link
Contributor Author

robbanl commented Nov 2, 2023

@duncanmcclean Thank you for this, and of course, I won't commit the main/master, I guess I was in a hurry and forgot to create my own branch :D Glad you solved this anyhow! Much appreciated!

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.

None yet

2 participants