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: Support non-english languages for Signup errors #4474

Merged
merged 2 commits into from
Aug 14, 2023

Conversation

g123k
Copy link
Collaborator

@g123k g123k commented Aug 7, 2023

Hi everyone,

The Dart package parses the content of the webpage to find errors during the signup.
However the different messages are translated, hence a partially working feature.

So instead of showing that kind of error:
Screenshot_1691411774

It will now show:
Screenshot_1691412088

That's could be better, but for the upcoming release, it will be OK.

@g123k g123k added the 👥 User management Account login, signup, signout label Aug 7, 2023
@g123k g123k self-assigned this Aug 7, 2023
@g123k g123k requested a review from a team as a code owner August 7, 2023 12:42
@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2023

You can test this PR on: Android

@teolemon
Copy link
Member

teolemon commented Aug 7, 2023

info • Unnecessary escape in string literal • packages/smooth_app/lib/pages/user_management/sign_up_page.dart:378:34 • unnecessary_string_escapes
info • Unnecessary escape in string literal • packages/smooth_app/lib/pages/user_management/sign_up_page.dart:378:41 • unnecessary_string_escapes

@teolemon teolemon merged commit 023d8e5 into openfoodfacts:develop Aug 14, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sign-Up 👥 User management Account login, signup, signout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error messages for registration should be improved for non-English languages
2 participants