1201909 - Link to UI Tour from About:Privatebrowsing built-in page

Status: VERIFIED FIXED

(Firefox :: General, defect, P1)

Description

[Javaun Moradi [:javaun]]

The about:privatebrowsing blue button "See how this works" currently links to a SUMO page. 

As soon as the UI tour is live in production, we'll want to link to it and uplift. Bgrins said ideally we land this patch and uplift to 42 Aurora before 42 cuts to Beta on 9/21. 

The engagement team will have the UI tour live by 9/14, which gives us the minimum cushion of safety we need. If they can the tour to prod earlier, that's even better for us.

Comment 1

[Alex Gibson [:agibson]]

Bug 1190447 (TP Tour development has been merged into master). It will go to production a little later today in the next push as soon as someone is available. I will update here once it's live.

Comment 2

[Alex Gibson [:agibson]]

Note: once the tour goes live, the value for the `privacy.trackingprotection.introURL` preference should be updated to:

"https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/tracking-protection/start/"

Comment 3

[Alex Gibson [:agibson]]

The tour is now on production, thanks all!

Comment 4

[Matthew N. [:MattN]]

Attachment: MozReview Request: Bug 1201909 - Set the production www.mozilla.org URL for the tracking protection tour. r=javaun

Bug 1201909 - Set the production www.mozilla.org URL for the tracking protection tour. r=javaun

Comment 5

[Javaun Moradi [:javaun]]

Comment on attachment 8663188 [details]
MozReview Request: Bug 1201909 - Set the production www.mozilla.org URL for the tracking protection tour. r=javaun

https://reviewboard.mozilla.org/r/19735/#review17761

Looks good.

Comment 6

[Matthew N. [:MattN]]

Comment on attachment 8663188 [details]
MozReview Request: Bug 1201909 - Set the production www.mozilla.org URL for the tracking protection tour. r=javaun

Approval Request Comment
[Feature/regressing bug #]: PB Tracking protection in 42
[User impact if declined]: Tour button will link to SUMO and not the interactive tour on www.mozilla.org
[Describe test coverage new/current, TreeHerder]: None
[Risks and why]: Extremely low risk URL change
[String/UUID change made/needed]: None

Comment 7

[Javaun Moradi [:javaun]]

Thanks. This needs to be in 42 beta build 1 that GTB on MOnday 9/21 and distributes on 9/24.

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/372489a1e361

Comment 9

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8663188 [details]
MozReview Request: Bug 1201909 - Set the production www.mozilla.org URL for the tracking protection tour. r=javaun

Approved for uplift so tracking protection tour will point to the correct version once 42 goes to beta

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

This may be good to verify on a couple of different locales in early beta and then again when 42 goes to RC.

Comment 11

[Matthew N. [:MattN]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/b28fa60333bf

Comment 12

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/372489a1e361

Comment 13

[Petruta Horea [:phorea], Desktop QA]

Checked some locales with Firefox 42 beta 1 under Win 7 64-bit and Mac OS X 10.9.5.
Logged 1207545 and 1207643 for the issues found.

Encountered some locales (eg pl) where the panels are not translated. Should I fill this and track separately?

Comment 14

[Alex Gibson [:agibson]]

> Encountered some locales (eg pl) where the panels are not translated. Should I fill this and track separately?

No need to file bugs for locales that are not yet translated. We only enable locales as the translations become completed. Thanks

Comment 15

[Petruta Horea [:phorea], Desktop QA]

Thanks :agibson!

Marking as verified based on above comments.

Comment 16

[Paul Silaghi, QA [:pauly]]

Verified fixed FF 43.0a2 (2015-10-13) Win 7