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

feat: make nutrient search accentuation insensitive #3792

Merged
merged 5 commits into from
Mar 21, 2023

Conversation

Sudhanva-Nadiger
Copy link
Contributor

What

  • if search term contains accentuation letter the filtered result wont recognise that
  • now its been removed

Fixes bug(s)

Part of

Copy link
Contributor

@monsieurtanuki monsieurtanuki left a comment

Choose a reason for hiding this comment

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

Hi @Sudhanva-Nadiger!
Please have a look at my comments.

Copy link
Contributor

@monsieurtanuki monsieurtanuki left a comment

Choose a reason for hiding this comment

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

Thank you @Sudhanva-Nadiger, that looks good!
Could you please add two screenshots, with the app language in French, where you're looking for nutriments cafe and vitaminé?

@Sudhanva-Nadiger
Copy link
Contributor Author

Yes, I will do that !

@Sudhanva-Nadiger
Copy link
Contributor Author

@monsieurtanuki

@monsieurtanuki monsieurtanuki merged commit cd61afa into openfoodfacts:develop Mar 21, 2023
@monsieurtanuki
Copy link
Contributor

Thank you @Sudhanva-Nadiger, that's perfect!

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

Successfully merging this pull request may close these issues.

Nutrient search is accentuation sensitive
2 participants