insights percuto newsletter form has no javascript check for required fields

Bug #1660616 reported by Peter Mahnke on 2017-01-31
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Insights
High
Graham Bancroft

Bug Description

Hi guys

I came across a few leads with no email addresses that filled out the insights subs form.. Now I can see that it's not a required field from Marketo side, but using js this was made as required on the page correct?

This list: https://app-sjg.marketo.com/#SL29650928B2 has over 300 people that filled that form but have no email, how is this possible since its a required field? Is this form implemented somewhere else?

I know this form was also used for cloud subs previously in the footer but AFAIK this was replaced already, but when I reduced the data of activity of people who filled the form and email address is blank in the last 15 days, I still get 11 leads.

I'm not sure how to investigate further, as this is not a marketo form per se Im not sure if Marketo support would look into it.

Cheers,
Lucia

Graham, I think the form is missing the bit that checks on submit.

Related branches

David Callé (davidc3) wrote :

After some testing, note that this only happens on insights.canonicalwebteam.com

David Callé (davidc3) wrote :

Disregard the previous comment, while it's correct that insights.canonicalwebteam.com doesn't do any validation on the email field, it seems it's not the only cause for this report, as some leads without an address are coming from insights.ubuntu.com

Lucia de Castro (luciadecastro) wrote :

Form on insights.ubuntu.com doesn't validate e-mail format, so it's possible to submit just with a space in that field. If we could please have a proper email validation (i.e. <email address hidden>) for this field would be fab. Thanks!

Lucia de Castro (luciadecastro) wrote :

Hi guys

How is this one going? Still getting empty emails :(

Hi Lucia,

I'm working in this right now, should have a fix in soon.

Graham

Lucia de Castro (luciadecastro) wrote :

Thank you :)

Hi Lucia,

This is fixed and should be live at some point today.

All the best,

Graham

Changed in resource-centre:
status: Triaged → Fix Committed
Changed in resource-centre:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers