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

[Experiment] The “Must be a valid email.” error message of the “Enter Email Address” field is not displayed if the checkbox is ticked #1607

Closed
heres-maria-sv-inactive opened this issue Mar 23, 2020 · 2 comments
Assignees

Comments

@heres-maria-sv-inactive

[Affected Versions]:

  • Firefox Release 74.0, Build ID 20200309095159
  • Firefox Beta 75.0b7 Build ID 20200322132212
  • Firefox Preview (Fenix) 4.1.0 Build #207711637

[Affected Platforms]:

  • Windows 10 (x64)
  • Mac 10.15.3
  • Motorola One Vision - Android 10

[Prerequisites]:

[Steps to reproduce]:

  1. Enter an invalid input in the “Enter Email Address” field (e.g. 123).
  2. Click the “Check for Breaches” button.

[Expected results]:

  • The “Must be a valid email.” error message is displayed and the “Enter Email Address” field is highlighted in a red outline.
  • The FxA sign-in process is not triggered.

[Actual results]:

  • No error message is displayed and the “Enter Email Address” field is not highlighted.
  • The FxA sign-in process is triggered.

[Notes]:

  • This issue is also reproducible for Version B: Opt In, if the checkbox is ticked.
  • This issue is not reproducible for Version A: Control.
  • Attached is a recording of the issue.

monitor front-end validation

@maxxcrawford
Copy link
Contributor

@heres-maria Added a fix for this. Please verify on the test site.

@heres-maria-sv-inactive
Copy link
Author

I have verified that this issue is no longer reproducible using Firefox Release 74.0 Build ID 20200309095159 and Chrome v. 80.0.3987.149 on Windows 10 (x64) as well as Firefox Preview 4.1.0 Build #20771637 on Android 10 (Motorola One Vision).
The "Must be a valid email" error message is now displayed.
As a result, I am closing this issue.

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

No branches or pull requests

2 participants