Closed Bug 1035846 Opened 10 years ago Closed 10 years ago

ToS notice in Loop popup window points to wrong URL

Categories

(Hello (Loop) :: Client, defect, P1)

defect
Points:
1

Tracking

(firefox34 fixed, firefox35 fixed)

RESOLVED FIXED
mozilla35
Iteration:
35.2
Tracking Status
firefox34 --- fixed
firefox35 --- fixed

People

(Reporter: RT, Assigned: standard8)

References

Details

(Whiteboard: [loop-uplift])

Attachments

(2 files)

The current ToS notice points to https://accounts.firefox.com/en-us/legal/terms The Loop ToS are on https://github.com/mozilla/legal-docs/blob/master/WebRTC_ToS/en-US.md ALthough we're awaiting confirmation about the URL to use. This bug is to track: * Confirmation of URL pointing to Loop ToS * Implementation on the Loop desktop client
Shell please confirm the URL when you get the info.
Flags: needinfo?(sescalante)
Whiteboard: --do_not_change-- [mozilla33 carry over]
Target Milestone: mozilla33 → mozilla34
Whiteboard: --do_not_change-- [mozilla33 carry over] → [mozilla33 carry over]
Flags: needinfo?(sescalante)
Whiteboard: [mozilla33 carry over] → [p=1, mozilla33 carry over]
What is the status of the TOS URL? Has it been implemented? It is hard to know from the actions/comments in this bug and just following up to make sure we get this done asap.
Geoff, this is to be handled in bug 1044411. We need to know where we should host this still.
Flags: needinfo?(sescalante)
Alexis -- Do you need any client work on this bug, or can I close this as a dup of bug 1044411 if that's where all the work for this is being done?
Flags: needinfo?(alexis+bugs)
We don't know where this will be stored yet. bug 1044411 is the one where this decision will be done, and then client needs to be updated with this url after that.
Flags: needinfo?(alexis+bugs)
Alexis -- So, all the work will be done on bug 1044411 and then this bug is just for updating the URL. Am I correct?
Flags: needinfo?(alexis+bugs)
This is correct, yes.
Flags: needinfo?(alexis+bugs)
Depends on: 1044411
Whiteboard: [p=1, mozilla33 carry over] → [p=1, mozilla33 carry over][qa+]
Arcadio approved URL, so confirming this is unblocked to use https://hello.firefox.com/legal/terms as the ToS URL.
Flags: needinfo?(sescalante)
Adam is handling this over on bug 1044411
Assignee: nobody → adam
Whiteboard: [p=1, mozilla33 carry over][qa+] → [p=1, mozilla33 carry over][qa+][loop-uplift]
Target Milestone: mozilla34 → mozilla35
Now bug 1044411 has landed, this updates the ToS link for the standlone client. I'm proposing we change the desktop one separately, once bug 1044411 has been pushed to production.
Attachment #8490763 - Flags: review?(dmose)
Assignee: adam → standard8
Iteration: --- → 35.2
Comment on attachment 8490763 [details] [diff] [review] Update the ToS link for the Loop standalone UI. Review of attachment 8490763 [details] [diff] [review]: ----------------------------------------------------------------- r=dmose
Attachment #8490763 - Flags: review?(dmose) → review+
Points: --- → 1
Keywords: leave-open
Whiteboard: [p=1, mozilla33 carry over][qa+][loop-uplift] → [qa+][loop-uplift][fix desktop when bug 1044411 is on production]
QA Contact: anthony.s.hughes
Flags: qe-verify+
Whiteboard: [qa+][loop-uplift][fix desktop when bug 1044411 is on production] → [loop-uplift][fix desktop when bug 1044411 is on production]
Mark -- can we mark this as resolved now?
Flags: needinfo?(standard8)
(In reply to Maire Reavy [:mreavy] (Plz needinfo me) from comment #16) > Mark -- can we mark this as resolved now? No, we need to desktop once production is live.
Flags: needinfo?(standard8)
I had to push a follow-up fix so that the standalone client works correctly with the ToS on the staging server: https://hg.mozilla.org/integration/fx-team/rev/56e6f150cd6c
Now we've got the page live, here's the link change for the panel.
Attachment #8495351 - Flags: review?(adam)
Comment on attachment 8495351 [details] [diff] [review] Update the ToS link for the Loop panel. Review of attachment 8495351 [details] [diff] [review]: ----------------------------------------------------------------- LGTM
Attachment #8495351 - Flags: review?(adam) → review+
Keywords: leave-open
Whiteboard: [loop-uplift][fix desktop when bug 1044411 is on production] → [loop-uplift]
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
I'm untracking this for verification. This will be covered through smoketesting.
Flags: qe-verify+ → qe-verify-
Comment on attachment 8490763 [details] [diff] [review] Update the ToS link for the Loop standalone UI. Approval Request Comment Uplift request for patches staged and tested on Fig
Attachment #8490763 - Flags: approval-mozilla-aurora?
Attachment #8495351 - Flags: approval-mozilla-aurora?
Comment on attachment 8490763 [details] [diff] [review] Update the ToS link for the Loop standalone UI. I worked with Randell and Maire on uplifting a large number of Loop bugs at once. All of the bugs have been staged on Fig and tested by QE before uplift to Aurora. As well, all of the bugs are isolated to the Loop client. Randell handled the uplift with my approval. I am adding approval to the bug after the fact for bookkeeping.
Attachment #8490763 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #8495351 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
We have a Moztrap smokestest for ToS/Privacy in the Hello panel and Standalone page.
Flags: in-moztrap+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: