Closed Bug 1463821 Opened 6 years ago Closed 6 years ago

Setup WNP for users coming from <61.0 and receiving the 61.0 release

Categories

(Release Engineering :: Release Requests, defect)

defect
Not set
normal

Tracking

(firefox61blocking verified)

VERIFIED FIXED
Tracking Status
firefox61 blocking verified

People

(Reporter: erenaud, Assigned: craigcook)

References

()

Details

Attachments

(9 files)

Request is to have the product point to the /whatsnew page in the Firefox 61.0 release and show it (WNP). 

Locales are TBD but are plan is to have final copy in a PR on May 29th allowing ample time for l10n community to localize in time for the release date.
Assignee: nobody → craigcook.bugz
If the user is not signed in, we show the Firefox Account signup (the form will appear in the white box).
If the user is signed in but has not synced to one or more mobile devices, we promote Firefox for mobile.

If they are in a locale that supports the send-to-device widget, we show the widget. If they are also in a country that supports SMS, we offer the SMS option. Other countries only get email.
If the user is signed in but hasn't synced to a mobile device, we promote Firefox for mobile. 

If their locale doesn't support the send widget, we show a QR code.
If the user is signed in and has synced a mobile device, we promote Firefox Focus.

If their locale supports the send widget, we show the widget. If they are in a country that supports SMS, we offer the SMS option (other countries only get email).
If the user is signed in and has synced a mobile device, we promote Firefox Focus.

If their locale doesn't support the send widget, we show a QR code.
See Also: → 1462941
Lauren - while the QR codes will be updated the copy and design is in place and ready for QA review/signoff by the Team Lead. Could you please do so and provide comments by EOD tomorrow, Tuesday June 19th?
Flags: needinfo?(lniolet)
Flags: needinfo?(craigcook.bugz)
Flod - can you please review the list of locales that have localized the new content and provide that here?

Please include zh-TW and id, regardless of the status in the lang file, as both of those pages are hard coded templates and need to be shown that content.
Flags: needinfo?(francesco.lodolo)
@mihai
Here's the list.

Complete locales (46):
cak, cy, da, de, dsb, en-US, es-AR, es-CL, es-ES, es-MX, et, fa, fi, fr, fy-NL, gn, gu-IN, hsb, hu, ia, id, it, ja, ja-JP-mac, ka, kab, ko, lij, lt, ms, nb-NO, nl, nn-NO, pl, pt-BR, pt-PT, sk, sl, sq, sr, sv-SE, tr, uk, vi, zh-CN, zh-TW

{
    "locales": [
        "cak",
        "cy",
        "da",
        "de",
        "dsb",
        "en-US",
        "es-AR",
        "es-CL",
        "es-ES",
        "es-MX",
        "et",
        "fa",
        "fi",
        "fr",
        "fy-NL",
        "gn",
        "gu-IN",
        "hsb",
        "hu",
        "ia",
        "id",
        "it",
        "ja",
        "ja-JP-mac",
        "ka",
        "kab",
        "ko",
        "lij",
        "lt",
        "ms",
        "nb-NO",
        "nl",
        "nn-NO",
        "pl",
        "pt-BR",
        "pt-PT",
        "sk",
        "sl",
        "sq",
        "sr",
        "sv-SE",
        "tr",
        "uk",
        "vi",
        "zh-CN",
        "zh-TW"
    ]
}
Flags: needinfo?(francesco.lodolo) → needinfo?(mtabara)
Please also see the 6 Adjust Links requested:

Firefox, Focus and Klar Whatsnew 61 links with Apple CTA.

1) Firefox with Apple CTA: https://mzl.la/2K035Mm
2) Firefox Focus with apple CTA:https://mzl.la/2I1I2r5
3) Firefox Klar with apple CTA:https://mzl.la/2JSz6da

Firefox, Focus and Klar Whatsnew 61 general links (do not have apple CTA hardcoded, will open in Google play store for desktop)


1) Firefox: https://mzl.la/2M6EXbI
2) Firefox Focus: https://mzl.la/2I0kjYd
3) Firefox Klar: https://mzl.la/2tabw1i
Commits pushed to master at https://github.com/mozmeao/www-config

https://github.com/mozmeao/www-config/commit/29a1a4f12be6b2ff95a519bc227086a37dfcc9dd
[bug 1463821] Add SMS countries for Fx61 whatsnew page

https://github.com/mozmeao/www-config/commit/919e6bd9cd58633d4753abeef1e1241be00b6eaa
Merge pull request #112 from craigcook/master-whatsnew61

[bug 1463821] Add SMS countries for Fx61 whatsnew page
(In reply to Francesco Lodolo [:flod] from comment #13)
> @mihai
> Here's the list.
> 
> Complete locales (46):
> cak, cy, da, de, dsb, en-US, es-AR, es-CL, es-ES, es-MX, et, fa, fi, fr,
> fy-NL, gn, gu-IN, hsb, hu, ia, id, it, ja, ja-JP-mac, ka, kab, ko, lij, lt,
> ms, nb-NO, nl, nn-NO, pl, pt-BR, pt-PT, sk, sl, sq, sr, sv-SE, tr, uk, vi,
> zh-CN, zh-TW
> 
> {
>     "locales": [
>         "cak",
>         "cy",
>         "da",
>         "de",
>         "dsb",
>         "en-US",
>         "es-AR",
>         "es-CL",
>         "es-ES",
>         "es-MX",
>         "et",
>         "fa",
>         "fi",
>         "fr",
>         "fy-NL",
>         "gn",
>         "gu-IN",
>         "hsb",
>         "hu",
>         "ia",
>         "id",
>         "it",
>         "ja",
>         "ja-JP-mac",
>         "ka",
>         "kab",
>         "ko",
>         "lij",
>         "lt",
>         "ms",
>         "nb-NO",
>         "nl",
>         "nn-NO",
>         "pl",
>         "pt-BR",
>         "pt-PT",
>         "sk",
>         "sl",
>         "sq",
>         "sr",
>         "sv-SE",
>         "tr",
>         "uk",
>         "vi",
>         "zh-CN",
>         "zh-TW"
>     ]
> }

This is looking good and is reflected currently in the 61.0 RC WNP
Flags: needinfo?(mtabara)
Stefan - 

The updated WNP page for 61 is currently on demo here and ready for testing. Apologies for the delay here.

https://www-demo4.allizom.org/de/firefox/61.0/whatsnew/
Flags: needinfo?(stefan.georgiev)
We tested today WNP on 61 using release-localtest update channel and we found two possible issue:

1. updating 60.0.* to 61.0 without being signed in FxA, we receive this page: https://drive.google.com/open?id=1P1cp5-y4TLNN1iA5dm1EAemnjqWVhy2h.
I would had expected to receive the login to FxA page. Does WNP not apply for updates from 60 to 61?

2. updating 57.0.3 (fr) to 61.0 after login to FxA on both desktop and mobile results in this page: https://screenshots.firefox.com/HtDhMgAe6Pd3lrH6/www.mozilla.org 
I would had expected to receive the WNP with Firefox Focus.
Flags: needinfo?(erenaud)
Gah, we did not try on https://www-demo4.allizom.org/de/firefox/61.0/whatsnew/. Letting stefan &team take care of this.
We finished testing WNP on 61.0 using release-cdntest update channel and we faced the same issue as mentioned in comment #18.

1. (Linux) updating 60.0.1 → 61.0 with being signed in FxA, we received this page: https://drive.google.com/file/d/1P1cp5-y4TLNN1iA5dm1EAemnjqWVhy2h/view

We expect to receive page like https://bug1463821.bmoattachments.org/attachment.cgi?id=8983499
Flags: needinfo?(stefan.georgiev)
 (In reply to Stefan [:StefanG_QA] from comment #20)
> We finished testing WNP on 61.0 using release-cdntest update channel and we
> faced the same issue as mentioned in comment #18.
> 
> 1. (Linux) updating 60.0.1 → 61.0 with being signed in FxA, we received this
> page: https://drive.google.com/file/d/1P1cp5-y4TLNN1iA5dm1EAemnjqWVhy2h/view

Correct - User is signed into FxA without multiple devices detected -> users sees Fx mobile CTA (widget or QA code depending on locale)

> 
> We expect to receive page like
> https://bug1463821.bmoattachments.org/attachment.cgi?id=8983499

The Focus CTA is shown to users logged into FxA with multiple devices synced (not just 'logged in')
The Klar CTA is shown to users logged into FxA with multiple devices synced AND who are geolocated in Germany/Austria/Switzerland.
Flags: needinfo?(erenaud)
(In reply to Francesco Lodolo [:flod] from comment #13)
> @mihai
> Here's the list.
> 
> Complete locales (46):
> cak, cy, da, de, dsb, en-US, es-AR, es-CL, es-ES, es-MX, et, fa, fi, fr,
> fy-NL, gn, gu-IN, hsb, hu, ia, id, it, ja, ja-JP-mac, ka, kab, ko, lij, lt,
> ms, nb-NO, nl, nn-NO, pl, pt-BR, pt-PT, sk, sl, sq, sr, sv-SE, tr, uk, vi,
> zh-CN, zh-TW
> 
> {
>     "locales": [
>         "cak",
>         "cy",
>         "da",
>         "de",
>         "dsb",
>         "en-US",
>         "es-AR",
>         "es-CL",
>         "es-ES",
>         "es-MX",
>         "et",
>         "fa",
>         "fi",
>         "fr",
>         "fy-NL",
>         "gn",
>         "gu-IN",
>         "hsb",
>         "hu",
>         "ia",
>         "id",
>         "it",
>         "ja",
>         "ja-JP-mac",
>         "ka",
>         "kab",
>         "ko",
>         "lij",
>         "lt",
>         "ms",
>         "nb-NO",
>         "nl",
>         "nn-NO",
>         "pl",
>         "pt-BR",
>         "pt-PT",
>         "sk",
>         "sl",
>         "sq",
>         "sr",
>         "sv-SE",
>         "tr",
>         "uk",
>         "vi",
>         "zh-CN",
>         "zh-TW"
>     ]
> }

Per https://bugzilla.mozilla.org/show_bug.cgi?id=1455010#c20, `en-GB` and 'en-ZA' should provide the WNP with the device widget. Testing these 2 locales on release-cdntest, they DO NOT provide the WNP at all. ALso, the above mentioned locales are not contained in the complete list from comment 13. Any thoughts about this?
Flags: needinfo?(erenaud)
(In reply to Iulia Cristescu, QA [:JuliaC] from comment #23)
> Per https://bugzilla.mozilla.org/show_bug.cgi?id=1455010#c20, `en-GB` and
> 'en-ZA' should provide the WNP with the device widget. 

The page is not localized, therefore not activated. I'm looking at the linked comment, and it only says that these locales have a sendto widget (if a page is enabled), but I don't see anywhere that they MUST have a WNP page enabled.

> Testing these 2 locales on release-cdntest, they DO NOT provide the WNP at all. ALso, the
> above mentioned locales are not contained in the complete list from comment
> 13. Any thoughts about this?

The list of locales is based on the page being localized or not (which, for English variants, is a bit weird). en-GB and en-ZA have never been a requirement, as far as I'm concerned.
Per IRC discussion with Eric, we're done here.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(lniolet)
Flags: needinfo?(erenaud)
Flags: needinfo?(craigcook.bugz)
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: