Required field, even filling in all the fields.



5 years ago
5 years ago


(Reporter: rdrgwtrs, Assigned: dpoirier)


Firefox Tracking Flags

(Not tracked)


(Whiteboard: [kb=1093841] , URL)


(2 attachments)



5 years ago
Created attachment 794623 [details]
Captura de tela de 2013-08-23 09:17:20.png

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:23.0) Gecko/20100101 Firefox/23.0 (Beta/Release)
Build ID: 20130807180628

Steps to reproduce:

Fill all fields

Actual results:

show message : This field is required.

Expected results:



5 years ago
Locale: pt-BR / Portuguese (Brazilian)
Group: websites-security

Comment 1

5 years ago
Created attachment 794725 [details]
Screenshot of filled in fields and error

Confirmed that this is an issue. (see attached screenshot)

Mike - can you assign this to the right person to fix?


5 years ago
Assignee: nobody → malexis


5 years ago
Whiteboard: [kb=1093841]

Comment 2

5 years ago
Jess: This is probably due to this particular page not displaying the language field, so it's not getting filled in. Is there any reason at this point not to display country and language fields on all the m.o newsletter signup pages?
Flags: needinfo?(jdavis)

Comment 3

5 years ago
We'll need a language drop-down in the semi-near future (when we're ready to launch Spanish) - right now this newsletter is in EN only. 

The way it was working before is that either there was a blank language_iso2 passed through (thus preserving any Firefox & You non-EN language), or we explicitly passed through EN, but made it so that if the language there before was non-EN, we wouldn't overwrite it; these actions happened without the language field showing.

If we wanted to add in the Language field, It would have to show only EN at this time for the choice.  And we'd want to make sure that the logic is in place that if they have a non-EN language choice in Exact Target, we do not overwrite it with "EN". (ie if they had been subscribed to the ES Firefox & You newsletter, I want them to still get the "ES" version even if they signed up for an "EN" only newsletter).

Does this help?
Flags: needinfo?(jdavis)

Comment 4

5 years ago
Okay, since this is Priority 1, for now we'll just fix the page so it works again.

Pull request opened:
Assignee: malexis → dpoirier

Comment 5

5 years ago
Commits pushed to master at
BUg 908633 - Don't require lang field on newsletter signup
Bug 908633 - mark lang field required in HTML

If lang field is displayed, mark it required in HTML
Merge pull request #1213 from dpoirier/bug-908633-required-lang-field

Bug 908633 - Don't require lang field on newsletter signup

Comment 6

5 years ago
Ready for test at

(works for me)
Ever confirmed: true

Comment 7

5 years ago
Thanks Dan!

Raymond, do you have time to test this today/tomorrow perchance?

Test 1
Go to:
Try to signup
Make sure it is successful and you receive the welcome email

Test 2
Go to:  (it will be in English, but the language field will be blank)
Try to signup with a blank language
Make sure it errors and requires you to select a language.
Select a language and signup
Make sure you receive the subscription confirmation email in the correct language.
verified all steps

Test 1
verified all steps
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 9

5 years ago
Thanks Raymond!

Dan - let's go to prod!

Comment 10

5 years ago
Jess: your wish is our command. It looks like this was included in yesterday's prod push and is live.

Comment 11

5 years ago
Dan: Thanks! I just tested and everything looks good. Thank you for alerting us to this issue!

Comment 12

5 years ago
Thank you guys!
See you at the Summit :)

Comment 13

5 years ago
Sweet! We'll see you there! :D
You need to log in before you can comment on or make changes to this bug.