3 ms·
Great idea. Just a heads up, the email validation in your index.js isn't functioning; it accepts any arbitrary string (or none at all).
by brimtown 12y ago
Great idea. Just a heads up, the email validation in your index.js isn't functioning; it accepts any arbitrary string (or none at all).
- frandroid 12y agoIt's not broken; it's an intentional form of user validation.
- brimtown 12y agoIt's marked as an email field, and if you enter a random string, it replies "Message sent!". Not trying to be pedantic or critical, I just figured he'd want to know if his goal was actually to gather emails.
- milesokeefe 12y agoThanks for letting me know, I always appreciate input like this. Although in this case, not validating the emails client-side just means I have to validate emails after I receive them. Which is actually good in a way, I like seeing when people send stuff like "DROPTABLEuser" or "'SHOWPROCESSLIST'1=1", it's kind of a fun curiosity I suppose.
- milesokeefe 12y agoI didn't bother with any validation client-side, I have the email validation happen on the back end only. In retrospect, I should have placed the inputs in an actual <form> element so that Chrome's email validation would take effect.
- brimtown 12y agoNo worries! As someone only a couple years older than you, I have to say, congrats on making the move out there. It seems like you've got a lot of passion and commitment; wish I was in the Berkeley to take you up on the coffee.