AnomalyInnovations / serverless-stack-demo-client

Source for the demo app client in the Serverless Stack Guide
https://demo2.serverless-stack.com
MIT License
635 stars 204 forks source link

HTML5 input validation in Chrome prevents login #1

Closed marcusmolchany closed 7 years ago

marcusmolchany commented 7 years ago

The email input type in Login.js prevents the form from being submitted for usernames in Chrome.

screen shot 2017-03-12 at 10 47 09 pm
mfwhitman commented 7 years ago

As a quick workaround, I've just changed the FormControl attribute from type="email" to type="text"

jayair commented 7 years ago

So we DO want this field to be type="email" since we want users to login with their email as their username (as opposed to creating a separate username and making the process a bit more complicated).

But we are running into this issue in Chrome because in the Create a Cognito Test User chapter, we are directing folks to create a test user with a username instead of the email.

We'll fix that chapter shortly.

For now, if you already have a test user then go ahead and use the workaround @mfwhitman suggested (but I'm sure you guys figured that out already!).

jayair commented 7 years ago

Hey guys, we fixed the test user chapter and some other parts of the tutorial directing folks to create a test user with an email address instead of a username. So hopefully, people don't run into this issue anymore.

Here is the commit - https://github.com/AnomalyInnovations/serverless-stack-com/commit/29b748a4d98fd5b9a84252943621c3a4c456701c