-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Your email address
textfield needs validation
#25860
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
Comments
@stephendonner — are you thinking an error message something like "Failed to create support ticket. Check your email address." or similar? |
@rmcfadden3 was thinking we should validate the email address first, and throw an error message specific to its validation, before even trying to submit, actually. |
@stephendonner — so maybe something more like "Could not validate your email address. Please try again." or "Could not verify your email address. Please try again." instead? |
Sorry for the delay in responding; what @petemill implemented with the red error text (inline) LGMT 👍 |
The above requires |
Verified
|
Brave | 1.46.133 Chromium: 108.0.5359.71 (Official Build) (x86_64) |
---|---|
Revision | 1e0e3868ee06e91ad636a874420e3ca3ae3756ac-refs/branch-heads/5359@{#1016} |
OS | macOS Version 13.1 (Build 22C5050e) |
Steps:
- Installed
1.46.133
- launched Brave
- set up a
Brave VPN
connection viaaccount.bravesoftware.com
- clicked on the
VPN
button in the toolbar - clicked the
Gears
icon - clicked on
Contact technical support
Confirmed Your email address
text field shows correct validation message Please enter a valid email address for invalid email address
Confirmed validation message Please choose a reason
returned for not choosing a Subject
Confirmed validation message This field is required
returned for empty body
example | example | example | example |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Description
Your email address
textfield needs validationKudos to @MadhaviSeelam for the find!
Steps to Reproduce
1.45.90
Brave VPN
connectionVPN
button in the toolbarGears
iconContact technical support
aaaa
into theYour email address
fieldSubmit
Actual result:
Failed to create support ticket, check your connection.
Expected result:
Should balk at the non-email-address text
aaaa
and throw an error saying so, with theYour email address
textfield highlighted in redReproduces how often:
100%
Brave version (brave://version info)
Version/Channel Information:
N/A
Yes
Yes
Other Additional Information:
Miscellaneous Information:
cc @MadhaviSeelam @aguscruiz @bsclifton @spylogsster @rmcfadden3
The text was updated successfully, but these errors were encountered: