Skip to content

Conversation

@fitimvata
Copy link
Contributor

@fitimvata fitimvata commented Mar 25, 2023

Pull Request

I created a new pull request with changes that @bidoubiwa requested in pull request #1024.

Related issue

Fixes #539

What does this PR do?

  • Implement multiple sorting
  • Add tests
  • Update docs

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

@changeset-bot
Copy link

changeset-bot bot commented Mar 25, 2023

🦋 Changeset detected

Latest commit: 1e6fe28

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@meilisearch/instant-meilisearch Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@bidoubiwa
Copy link
Contributor

Incredible contribution 🔥🔥🔥 Thanks a lot.

Could you add a changeset ?

@bidoubiwa bidoubiwa added the enhancement New feature or request label Mar 27, 2023
@bidoubiwa bidoubiwa self-requested a review March 27, 2023 09:06
@fitimvata
Copy link
Contributor Author

@bidoubiwa Done!

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.

Thanks for the changeset :) I reviewed your code and its good! Could you also update the sortBy section in the readme by adding some info about this new usage?

@bidoubiwa bidoubiwa mentioned this pull request Mar 28, 2023
3 tasks
@fitimvata
Copy link
Contributor Author

fitimvata commented Mar 28, 2023

@bidoubiwa I have updated the readme.

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.

Thanks a lot 🙏 This is a very nice PR

@bidoubiwa
Copy link
Contributor

bors merge

@meili-bors
Copy link
Contributor

meili-bors bot commented Mar 29, 2023

@meili-bors meili-bors bot merged commit 69323f6 into meilisearch:main Mar 29, 2023
@fitimvata fitimvata deleted the add/support-for-multiple-sort-attributes branch March 29, 2023 09:12
@fitimvata
Copy link
Contributor Author

@bidoubiwa Thanks!

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.

SortBy should allow for multiple sort attributes at query time

2 participants