Update the copy for the cookie banner experiment
Categories
(Fenix :: Privacy, enhancement, P1)
Tracking
(firefox110 unaffected, firefox111 unaffected, firefox112 verified)
Tracking | Status | |
---|---|---|
firefox110 | --- | unaffected |
firefox111 | --- | unaffected |
firefox112 | --- | verified |
People
(Reporter: amejia, Assigned: amejia)
References
Details
(Whiteboard: [cookie-banner] [geckoview:m112])
Attachments
(3 files)
We want to update the copy for the cookie banner experiment with the new strings included here.
Assignee | ||
Updated•2 years ago
|
Comment 1•2 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 2•2 years ago
|
||
Comment on attachment 9322251 [details] [review]
[mozilla-mobile/firefox-android] Bug 1821468 - Update the copy for the cookie banner experiment (#1184)
Beta/Release Uplift Approval Request
- User impact if declined: We won't be able to run the experiment for cookie banner handling re-engagement dialog
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: With a clean version of Fenix
- Navigate to amazon.de
- Verify the new strings match the image attachment :
"Allow Firefox to reject cookie banners?
Firefox can automatically reject many cookie banner requests.
<Allow>
<Not now>
"
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Low as we are mainly updating strings
- String changes made/needed:
- Is Android affected?: Yes
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
Comment 4•2 years ago
|
||
Comment 5•2 years ago
|
||
Hello, just to clarify, the intention here is to land this for 112, does this bring an issue WRT our process?
Comment 6•2 years ago
|
||
Comment 7•2 years ago
|
||
Thanks for clarifying and clearing the request.
112 has not reached beta yet. 112 is still in nightly and the usual merge process applies.
I see the PR is queued to merge, so this will land for 112 without any need to uplift.
Updated•2 years ago
|
Comment 8•2 years ago
|
||
https://github.com/mozilla-mobile/firefox-android/commit/44b7e8b1c0f15a49b84d44042698a8efc63015d3
https://github.com/mozilla-mobile/firefox-android/commit/3c38a459619b6a4bb8786a2eb99d67956d2b2b4f
Assignee | ||
Comment 9•2 years ago
|
||
Thanks all for moving this forward and sorry for the confusion.
Comment 10•2 years ago
•
|
||
Verified as fixed on Nightly 112.0a1 from 12/03 with Xiaomi Pad5 (Android 12). The the copy for the cookie banner experiment is updated.
Updated•2 years ago
|
Comment 11•2 years ago
|
||
Assignee | ||
Comment 12•2 years ago
|
||
Thanks Adina!
Description
•