Closed Bug 1088636 Opened 9 years ago Closed 9 years ago

Desktop ToS url should use hello.firefox.com not call.mozilla.com

Categories

(Hello (Loop) :: Client, defect)

Product:
Hello (Loop)
Component:
Client
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox33 unaffected, firefox34+ fixed, firefox35+ verified, firefox36 verified)

Status:
VERIFIED FIXED
Milestone:
mozilla36
Iteration:
36.1
Tracking Flags:
Tracking Status
firefox33 --- unaffected
firefox34 + fixed
firefox35 + verified
firefox36 --- verified
Blocking Flags:
backlog Fx34+

People

(Reporter: standard8, Assigned: standard8)

Details

(Whiteboard: [landed in fx-team])

Attachments

(1 file)

Desktop ToS url should use hello.firefox.com not call.mozilla.com.
1.24 KB, patch
NiKo
: review+
lmandel
: approval-mozilla-aurora+
lmandel
: approval-mozilla-beta+
Details | Diff | Splinter Review 
We need to update the value in the preference file to use the new hello.firefox.com domain.
backlog: --- → Fx34+
Simple one-line patch.
Attachment #8511030 - Flags: review?(mdeboer)
Attachment #8511030 - Flags: review?(mdeboer) → review?(nperriault)
To verify:

- Open up Firefox Desktop
- Open the Loop panel
- If the ToS message isn't displayed:
-- Go into about:config
-- Reset "loop.seenToS"
-- Re-open the panel

Click the ToS link, it should navigate to https://hello.firefox.com/legal/terms/
Comment on attachment 8511030 [details] [diff] [review]
Desktop ToS url should use hello.firefox.com not call.mozilla.com.

Review of attachment 8511030 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=me.
Attachment #8511030 - Flags: review?(nperriault) → review+
Iteration: --- → 36.1
Whiteboard: [landed in fx-team]
Target Milestone: --- → mozilla36
Comment on attachment 8511030 [details] [diff] [review]
Desktop ToS url should use hello.firefox.com not call.mozilla.com.

Approval Request Comment
[Feature/regressing bug #]: Update ToS URL to match new location
[User impact if declined]: client would continue to fetch the terms of service from the old location
[Describe test coverage new/current, TBPL]: manually verified
[Risks and why]: No risk outside of Hello;  virtually no risk to Hello;  product and legal need us to use the latest URL location
[String/UUID change made/needed]: No strings
Attachment #8511030 - Flags: approval-mozilla-beta?
Attachment #8511030 - Flags: approval-mozilla-aurora?
[Tracking Requested - why for this release]:
The ToS URL location moved
tracking-firefox34: --- → ?
Flags: qe-verify-
Testing in 10/25 Nightly; looks good.
Sorry - "Tested", not Testing.  Verified on Nightly
Comment on attachment 8511030 [details] [diff] [review]
Desktop ToS url should use hello.firefox.com not call.mozilla.com.

Aurora+

The plan is to verify on Aurora on Sunday before uplift for beta4.
Attachment #8511030 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Verified on Aurora Nightly 10/26 Linux
Comment on attachment 8511030 [details] [diff] [review]
Desktop ToS url should use hello.firefox.com not call.mozilla.com.

Beta+
Attachment #8511030 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.