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

SI: Email Validation Could be Improved #421

Closed
mtrutledge opened this issue Oct 21, 2022 · 0 comments
Closed

SI: Email Validation Could be Improved #421

mtrutledge opened this issue Oct 21, 2022 · 0 comments
Labels
Milestone

Comments

@mtrutledge
Copy link
Contributor

Describe the bug

We had a user attempting to place an order on our site with a email address similar to the following:
mXX09871223_@example.com

Once she clicked to checkout she was told the email was invalid. She had already created a user in our system with that email address and was logged in. This is a weird email address, but it is still valid.

Software Versions

  • DNN: 09.10.06
  • Hotcakes: 03.06.00

To Reproduce

Steps to reproduce the behavior:

  1. Add items to the cart
  2. Go to checkout
  3. Try checking out as guest and enter email address mXX09871223_@example.com
  4. Hotcakes will stop the order and let the user know they need to enter a valid email address

Expected behavior

This email address should be considered valid and the order should be placed.

Actual behavior

The user was shown an error saying the email address was invalid.

@WillStrohl WillStrohl added this to the 03.07.00 milestone Oct 21, 2022
@WillStrohl WillStrohl removed this from the 03.07.00 milestone Oct 21, 2022
@WillStrohl WillStrohl changed the title Email Validation Could be Improved SI: Email Validation Could be Improved Oct 21, 2022
WillStrohl added a commit that referenced this issue Oct 24, 2022
…dation-Could-be-Improved

Updated the Regex method. Issue #421 Fixed
@WillStrohl WillStrohl added this to the 03.07.00 milestone Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants