Closed
Bug 2014083
Opened 5 months ago
Closed 5 months ago
Put custom review prompt back behind a feature flag
Categories
(Firefox for Android :: Experimentation and Telemetry, task)
Tracking
()
RESOLVED
FIXED
149 Branch
People
(Reporter: mkozinski, Assigned: mkozinski)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][group2])
Attachments
(2 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
Custom review prompt UI should be enabled only when a Nimbus feature flag is enabled. To simplify possible beta uplift add a new flag, say custom-review-prompt-ui and don't touch the existing flag (other than possibly updating docs and renaming for clarity).
Updated•5 months ago
|
| Assignee | ||
Comment 1•5 months ago
|
||
Pushed by mkozinski@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/56a37fb6f9fb
https://hg.mozilla.org/integration/autoland/rev/43d89a61b588
Create a new Nimbus flag for custom review prompt UI r=android-reviewers,twhite
Comment 3•5 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
status-firefox149:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch
Comment 4•4 months ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined: Without the flag, if custom review prompt hits issues again, we'll need to resort to uplifting to release to disable it.
- Code covered by automated testing: yes
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing:
- Risk associated with taking this patch: low
- Explanation of risk level: Standard Nimbus flag configuration
- String changes made/needed: no
- Is Android affected?: yes
Attachment #9544510 -
Flags: approval-mozilla-beta?
| Assignee | ||
Comment 5•4 months ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D281760
Updated•4 months ago
|
Attachment #9544510 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•4 months ago
|
status-firefox148:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•