-
Notifications
You must be signed in to change notification settings - Fork 798
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
Contact Form: Email validation regex needs updating #3734
Comments
Hi, just wondering if the decision was made to update this? I'm wondering if it is what is causing me to have so much trouble modifying the 'required' text and the text on the 'Submit' button on the contact form. Whatever is going on with the regex is causing the rest of the code to be unrecognized as JS in the editor. Thanks! |
@salomestarbuck We haven't made any changes to that feature yet.
It's worth noting that you can use the To change the submit button text, you can use a custom shortcode parameter, |
Thank you!!! On Mon, May 2, 2016 at 12:02 AM, Jeremy Herve notifications@github.com
Salomé |
So, would I place that code into the functions.php file of my theme? On Mon, May 2, 2016 at 7:06 AM, Salome Starbuck salomestarbuck@gmail.com
Salomé |
@salomestarbuck The The |
A sharp-eyed user has noticed that the email validation regex here:
https://github.com/Automattic/jetpack/blob/4.0.2/modules/contact-form/js/grunion.js#L502
... no longer matches the one specified here:
http://www.regular-expressions.info/email.html
Perhaps we should update? This also applies to WP.com's version of
grunion.js
.2635373-t
The text was updated successfully, but these errors were encountered: