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

VERIFIED FIXED in Firefox 34

Status

Hello (Loop)
Client
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
mozilla36
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

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

Details

(Whiteboard: [landed in fx-team])

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
We need to update the value in the preference file to use the new hello.firefox.com domain.
backlog: --- → Fx34+
(Assignee)

Comment 1

3 years ago
Created attachment 8511030 [details] [diff] [review]
Desktop ToS url should use hello.firefox.com not call.mozilla.com.

Simple one-line patch.
Attachment #8511030 - Flags: review?(mdeboer)
(Assignee)

Updated

3 years ago
Attachment #8511030 - Flags: review?(mdeboer) → review?(nperriault)
(Assignee)

Comment 2

3 years ago
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+
(Assignee)

Updated

3 years ago
Iteration: --- → 36.1
Whiteboard: [landed in fx-team]
Target Milestone: --- → mozilla36
(Assignee)

Comment 4

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/05b4f6cb488d
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: --- → ?
status-firefox33: --- → unaffected
status-firefox34: --- → affected
status-firefox35: --- → affected
status-firefox36: --- → affected
tracking-firefox34: ? → +
tracking-firefox35: --- → +
https://hg.mozilla.org/mozilla-central/rev/05b4f6cb488d
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
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+
https://hg.mozilla.org/releases/mozilla-aurora/rev/611859bcb5ef
status-firefox35: affected → fixed
Verified on Aurora Nightly 10/26 Linux
status-firefox35: fixed → verified
status-firefox36: affected → verified
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+
https://hg.mozilla.org/releases/mozilla-beta/rev/45d717da277d
status-firefox34: affected → fixed
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.