search

savvato-software / dtim-mobile

The application we use to interact with people during a tech interview meeting.
1 stars 3 forks source link

I'm New Here email field validator is not tight enough #5

Open haxwell opened 4 years ago

haxwell commented 4 years ago

The validator on the field will report a valid email address if you enter 'test@email' . It should report an error condition, and only report valid for truly valid email addresses. At a minimum, the current validator should also expect a period, and at least two characters, at the end of the string, in addition to what it currently expects. So 'test@email.co' would be considered valid.

haxwell commented 4 years ago

In the same way, you can enter fdsafdsa@fdsfads.com.dfd.sfdfasd.fdsa.fdsa.fdsa.fdsa.fsda and it considers that valid, too. It should not.