965867 - Update final URLs for Android Firefox Accounts landing

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Android Sync, defect, P1)

Description

[Nick Alexander :nalexander [he/him]]

We need to rev all of our URLs for final sign off: Old Firefox, Terms of Service, Privacy Notice, Forgot password?, etc.

Per discussion in Bug 965376, we're going to take these URLs away from localizers, so we can rev them freely.

Comment 1

[Ryan Feeley [:rfeeley]]

COPPA fail (aka Sorry Kiddo) Learn more link links to:
http://www.consumer.ftc.gov/articles/0031-protecting-your-childs-privacy-online

Wireframe: https://www.dropbox.com/s/eey1e27k8uzbys4/Sorry%20Kiddo.pdf

Comment 2

[Rob Lord [:rlord]]

Per 959849, "Using an older version of Sync?" links in Android should point at https://support.mozilla.org/1/mobile/%VERSION%/%OS%/%LOCALE%/old-sync

Comment 3

[Nick Alexander :nalexander [he/him]]

New code generates links that will look like: 

I FxAccounts(372)             fennec_nalexander :: FxAccountGetStartedActivity :: $$FxA PII$$: Old Firefox url is: https://support.mozilla.org/1/mobile/24.0a1/Android/en-US/old-sync

Comment 4

[Nick Alexander :nalexander [he/him]]

Attachment: github PR

This makes the strings look like:

<!-- Firefox Account links. -->
<!-- https://support.mozilla.org/1/mobile/%VERSION%/%OS%/%LOCALE%/old-sync -->
<string name="fxaccount_link_old_firefox">https://support.mozilla.org/1/mobile/&formatS1;/&formatS2;/&formatS3;/old-sync</string>
<string name="fxaccount_link_create_not_allowed">http://www.consumer.ftc.gov/articles/0031-protecting-your-childs-privacy-online</string>
<string name="fxaccount_link_tos">https://accounts.firefox.com/legal/terms</string>
<string name="fxaccount_link_pn">https://accounts.firefox.com/legal/privacy</string>
<string name="fxaccount_link_forgot_password">https://accounts.firefox.com/reset_password</string>

Comment 5

[Nick Alexander :nalexander [he/him]]

s/strings/URLs/

Comment 6

[Nick Alexander :nalexander [he/him]]

Note to lander: this needs formatS3 added to mobile/android/base/strings.xml.in.

Comment 7

[Richard Newman [:rnewman]]

Comment on attachment 8368118 [details] [review]
github PR

Nits on GH.

Comment 8

[Nick Alexander :nalexander [he/him]]

https://hg.mozilla.org/services/services-central/rev/85db73060d6b

Comment 9

[Richard Newman [:rnewman]]

https://hg.mozilla.org/mozilla-central/rev/85db73060d6b

Comment 10

[Ioana Chiorean]

The corrects links are implemented in the nightly build! Setting this as Verified Fixed.
PS: Some of the links are blanks - not having content

Device: Galaxy Nexus 
OS: Android 4.3
Build: 01/31

Comment 11

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8368118 [details] [review]
github PR

Done, landed, shipped, so clearing f? requests.