Merge pull request #8439 from mbauskar/subsciption-fixes

[minor] validate email address before get update button click
This commit is contained in:
Nabin Hait 2017-04-13 18:41:49 +05:30 committed by GitHub
commit 8c7eb444ef

View File

@ -12,7 +12,7 @@
frappe.ready(function() { frappe.ready(function() {
$("#footer-subscribe-button").click(function() { $("#footer-subscribe-button").click(function() {
if($("#footer-subscribe-email").val()) { if($("#footer-subscribe-email").val() && valid_email($("#footer-subscribe-email").val())) {
$("#footer-subscribe-email").attr('disabled', true); $("#footer-subscribe-email").attr('disabled', true);
$("#footer-subscribe-button").html("Sending...") $("#footer-subscribe-button").html("Sending...")
.attr("disabled", true); .attr("disabled", true);
@ -31,7 +31,7 @@ frappe.ready(function() {
}); });
} }
else else
frappe.msgprint(frappe._("Please enter email address")) frappe.msgprint(frappe._("Please enter valid email address"))
}); });
}); });
</script> </script>