Closed
Bug 1281902
Opened 8 years ago
Closed 8 years ago
Email opt in form does not appear for Spanish versions of account creation form
Categories
(Cloud Services :: Server: Firefox Accounts, defect)
Cloud Services
Server: Firefox Accounts
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bniolet, Assigned: stomlinson)
References
()
Details
(Whiteboard: [fxa])
Attachments
(3 files)
The email opt-in checkbox does not appear on Spanish language versions of the Account creation page. Likewise, Spanish language preference menu does not allow users to change their subscription preferences. We discovered in testing the new CRM setup that if the browser's language is set to any Spanish language (ex: es-es, es-mx, etc.) the opt-in checkbox does not show. The box appears and works correctly for other languages supported by email (RU, DE, PT-BR, etc., but not Spanish).
Reporter | ||
Comment 1•8 years ago
|
||
In Bug 1179873, we integrated the accounts creation form to allow email opt-ins for languages other than English. https://bugzilla.mozilla.org/show_bug.cgi?id=1179873
Reporter | ||
Comment 2•8 years ago
|
||
Just for comparison, other languages do have the check box available.
Comment 3•8 years ago
|
||
From the list here: https://github.com/mozilla/fxa-content-experiments/blob/dev/communication-prefs/communication-prefs.js#L12 It looks like the form is enabled for the base "es" locale, but not for any of the (many!) region-specific tags. Should we add a `es-[a-z]{2}` rule to match any Spanish locale, similar to the existing `en-[a-z]{2}` rule that matches any English locale?
Flags: needinfo?(bniolet)
Reporter | ||
Comment 4•8 years ago
|
||
I can confirm that if you set your language to just "es" you get the form. Knowing what I know about the Lat-Am / Es-Es communities though, no one will make that their language. I think a regex would be just perfect to solve this.
Flags: needinfo?(bniolet)
Assignee | ||
Updated•8 years ago
|
Whiteboard: [fxa]
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Comment 5•8 years ago
|
||
Issued a pull request to the fxa-content-experiments repo here: https://github.com/mozilla/fxa-content-experiments/pull/38
Assignee: nobody → stomlinson
Status: NEW → ASSIGNED
Comment 6•8 years ago
|
||
Thanks, Shane. Do you have an ETA for this to go live?
Flags: needinfo?(stomlinson)
Assignee | ||
Comment 7•8 years ago
|
||
A new train with this feature should go into production sometime this week. Shifting the ni to :jrgm and :jbuck who are in charge of our deployments.
Flags: needinfo?(stomlinson)
Flags: needinfo?(jrgm)
Flags: needinfo?(jbuckley)
Comment 8•8 years ago
|
||
Thanks, Shane! :jrgm & :jbuck - did this go live last week?
Comment 9•8 years ago
|
||
Yep, this is working correctly now
Flags: needinfo?(jrgm)
Flags: needinfo?(jbuckley)
Reporter | ||
Comment 10•8 years ago
|
||
Hi guys. I tested this and there's no email opt in check box on the form for any es-xx language. So es-cl or es-py the page has no check box. If you set the browser language to just "es" there is a check box. Am I missing something? Am I supposed to test this in a staging environment or is it supposed to be live? Steps to reproduce 1) launch Firefox profile manager. 2) Create a blank profile. 3) Start Firefox with the new profile. 4) In Preferences > Content, set the first language to es-py or es-mx or anything es-xx. 5) navigate to accounts.firefox.com. 6) The accounts page will not have an email opt-in box.
Flags: needinfo?(jrgm)
Flags: needinfo?(jbuckley)
Assignee | ||
Comment 11•8 years ago
|
||
When I test locally with es-AR, the box shows up correctly. This leads me to believe the wrong sha is being used for fxa-content-experiments on prod. The last branch is is `train-64` with sha 07922dc8021ec349e80e846273743923aa6ffdba, created on June 3rd. We need sha a1f8d718d545fe6782f7c4009562a727e6418768, created on June 30th. Keeping the ni for jbuck and jrgm.
Comment 12•8 years ago
|
||
This new commit came along with train-67, which just shipped. I tested with es and es-MX and confirmed that I get the opt-in checkbox for both now. Ben, want to confirm?
Flags: needinfo?(jrgm)
Flags: needinfo?(jbuckley)
Reporter | ||
Comment 13•8 years ago
|
||
Just checked with es-cl (Chile). It worked!! I'm confirming this is fixed.
Updated•8 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•