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

fix(type): getTypoTolerance return value type #1455

Merged

Conversation

riccox
Copy link
Contributor

@riccox riccox commented Mar 10, 2023

Pull Request

getTypoTolerance function should return type "TypoTolerance" rather than string array.

What does this PR do?

  • replace getTypoTolerance return value type with "TypoTolerance"

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!

getTypoTolerance function should return type "TypoTolerance" rather than string array.
@bidoubiwa
Copy link
Contributor

Hey @riccox thanks a lot for this contribution :)

Could you change the failing test in tests/typo_tolerance.test.ts on line 43 ?

@bidoubiwa bidoubiwa self-requested a review March 13, 2023 12:53
@bidoubiwa bidoubiwa added the bug Something isn't working label Mar 13, 2023
@bidoubiwa
Copy link
Contributor

Hey @riccox

It would be nice if we could merge this nice contribution :) Can you fix the test? You only have to remove the string[] part in the test and it should be ok.

If you have no time. I can merge this PR into my own PR

@riccox
Copy link
Contributor Author

riccox commented Mar 29, 2023

@bidoubiwa Sorry to be later.
Sure, you can merge this PR into your own PR.
I just have no much time for this recently.

@bidoubiwa bidoubiwa changed the base branch from main to fix_get_typo_tolerance_return_type June 13, 2023 09:56
@bidoubiwa bidoubiwa merged commit ac37a81 into meilisearch:fix_get_typo_tolerance_return_type Jun 13, 2023
@meilisearch meilisearch deleted a comment from meili-bot Jun 13, 2023
@riccox riccox deleted the patch-1 branch June 13, 2023 15:52
meili-bors bot added a commit that referenced this pull request Jun 14, 2023
1518: Fix get typo tolerance return type r=bidoubiwa a=bidoubiwa

Following #1455, this PR fixes the test of the PR.

Co-authored-by: Ricco Xie <[email protected]>
Co-authored-by: Charlotte Vermandel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants