Skip to content

Conversation

@kirimgan
Copy link

@kirimgan kirimgan commented Mar 6, 2023

Pull Request

Related issue

Fixes #539

What does this PR do?

  • This PR add multiple search parameters to the index string

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?

Thank you so much for contributing to Meilisearch!

@changeset-bot
Copy link

changeset-bot bot commented Mar 6, 2023

⚠️ No Changeset found

Latest commit: ca30bf6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@bidoubiwa
Copy link
Contributor

bidoubiwa commented Mar 14, 2023

Hey @kirimgan thanks a lot for contributing, unfortunately there are a lot of issues in your PR:

  • changes in package.json
  • linting issues
  • tests do not pass
  • no additional tests were added
  • code could use .map instead of for
  • sort code should be in its file and it own function

Could you fix them?

@bidoubiwa bidoubiwa added the enhancement New feature or request label Mar 14, 2023
@bidoubiwa
Copy link
Contributor

closed in favor of #1041

@bidoubiwa bidoubiwa closed this Mar 28, 2023
meili-bors bot added a commit that referenced this pull request Mar 29, 2023
1041: Add support for multiple sort attributes r=bidoubiwa a=fitimvata

# 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?
 - [x] Implement multiple sorting
 - [x] Add tests
 - [x] Update docs

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?



Co-authored-by: Fitim Vata <fitimvata28@gmail.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.

SortBy should allow for multiple sort attributes at query time

2 participants