Closed Bug 1870701 Opened 11 months ago Closed 9 months ago

Update the base privacy policy link and anchor text in the Review Checker opt-in screen

Categories

(Fenix :: Shopping, enhancement, P2)

enhancement

Tracking

(firefox121 wontfix, firefox122 wontfix, firefox123 wontfix, firefox124 verified, firefox125 fixed)

RESOLVED FIXED
125 Branch
Tracking Status
firefox121 --- wontfix
firefox122 --- wontfix
firefox123 --- wontfix
firefox124 --- verified
firefox125 --- fixed

People

(Reporter: asafko, Assigned: vdreghici, NeedInfo)

References

Details

(Whiteboard: [fxdroid] [fakespot-android-mvp])

Attachments

(3 files)

Hi team,
We have updated the Privacy Policy to clearly explain the minimal nature of data that's collected and processed during Review Checker use in Firefox. We now need to update the text and link in the paragraph that mentions privacy policy and terms of use to the following:

"By selecting "Yes, try it" you agree to Firefox's privacy notice and Fakespot's terms of use.

The link and anchor text is changing:

Terms of use link remains the same, but we are adding "Fakespot's" to note that two links belong to two different domains.

Blocks: 1870702
Severity: -- → N/A
Priority: -- → P2
Whiteboard: [fxdroid] [fakespot-android-mvp]

Ania says there is no need to rush this fix, so we don't need to uplift to a 121.x dot release.

Summary: Update the base privacy policy link in the Review Checker opt-in screen → Update the base privacy policy link and anchor text in the Review Checker opt-in screen

@Ania - Heard back from Localization.

Here are the strings we should use:

By selecting "Yes, try it" you agree to these items:
Firefox's privacy notice
Fakespot's terms of use

These are updated in the string inventory.

As before, these notes are important to include in the code for localization:

After the colon, what appears are two links, each on their own line. The first link is to a Privacy policy. The second link is to Terms of use. Placeholders should be used for the “Fakespot” and “Firefox” brands.

Assignee: nobody → Vlad.DreghiciPopa

Taking a look at this PR, considering that we are aiming to revert to the previous text, where we had both links in the text instead of two separate links:

After that patch lands, we would need to update the above text to match the new requirement in this ticket, like this:

Adding NI for double-checking and visibility.

LE: Added PR in which I attached an image of how the new text would look.

Flags: needinfo?(asafko)
Blocks: 1872419
Blocks: fenix-fakespot-integration
No longer blocks: 1870702
Blocks: 1870702
No longer blocks: 1872419

Hi Vlad,
Updating the links and anchor text and keeping the cohesive sentence makes sense to me (and IMO looks better than two separate links), let's keep it that way.

Flags: needinfo?(asafko)

Katie, just an FYI on this.

Flags: needinfo?(klower)
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
Flags: qe-verify+

Verified as fixed on the latest Nightly - 124.0a1 from 14.02.2024 with Motorola G9 plus (Android 11) and OnePlus A3000 (Android 6).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Status: VERIFIED → RESOLVED
Closed: 9 months ago9 months ago
Target Milestone: 124 Branch → 125 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: