-
Notifications
You must be signed in to change notification settings - Fork 152
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
Client-side email address validation makes unhelpful suggestions. #771
Comments
The library we use is https://github.com/mailcheck/mailcheck |
Copying @kanaabe's comment here mailcheck/mailcheck#139 This definitely looks like the issue. A PR seems to have been up for 3 months. Can someone npm instal from that person's branch and see if it indeed fixes the issue? |
@1aurabrown We should probably install mailcheck from the git repo and, if it does indeed fix the issue for us, ask the mailcheck maintainers to do a release with that fix. |
Should we switch or take ownership? |
From #766.
In this example, the suggestion doesn’t include a TLD and using it would still not make the field validate, so it seems like an unhelpful suggestion. We use this across the site, so improving it touches many places.
The text was updated successfully, but these errors were encountered: