Closed
Bug 1661803
Opened 5 years ago
Closed 5 years ago
Check for presence of default add-on before unlocking studies
Categories
(Firefox :: Pioneer, defect)
Tracking
()
RESOLVED
FIXED
82 Branch
People
(Reporter: javaun, Assigned: rhelmer)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
The required telemetry probe noting the timestamp of the user's acceptance of Pioneer enrollment consent is moving offtrains. This and other required functionality must be present before the user can join studies.
We are adding a check before the "Available Studies" are unlocked.
Requirement:
- The About Pioneer page already has a check to see if a user is enrolled before unlocking the Available Studies section
- Add a second check to ensure that the user has a default add-on before unlocking "Available studies"
Assignee | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Comment 2•5 years ago
|
||
Comment on attachment 9172789 [details]
Bug 1661803 - ensure default study is installed before unlocking studies r?maxxcrawford
Beta/Release Uplift Approval Request
- User impact if declined: User may be able to incorrectly install studies without installing the core addon. This bug reduces the possibility of that happening.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Only impacts about:pioneer. No risk to Firefox.
- String changes made/needed: None
Attachment #9172789 -
Flags: approval-mozilla-beta?
Pushed by rhelmer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c7a9b9d944a0
ensure default study is installed before unlocking studies r=fluent-reviewers,flod,sfoster
Comment 4•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
Comment 5•5 years ago
|
||
Comment on attachment 9172789 [details]
Bug 1661803 - ensure default study is installed before unlocking studies r?maxxcrawford
Approved for 81.0b5.
Attachment #9172789 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 6•5 years ago
|
||
bugherder uplift |
status-firefox81:
--- → fixed
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•