Closed
Bug 1865883
Opened 1 year ago
Closed 1 year ago
Add confirmation messages when setting the default mail-to handler
Categories
(Firefox :: Shell Integration, task)
Firefox
Shell Integration
Tracking
()
VERIFIED
FIXED
122 Branch
People
(Reporter: michaelahughesuk, Assigned: mpohle)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fidedi-mailto])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-beta+
|
Details | Review |
This is part of the phase 2 work for setting Firefox setting itself and sites as the default mail-to handler.
Reporter | ||
Updated•1 year ago
|
Assignee: nobody → mpohle
Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Pushed by mhughes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a2f4853e3d25
added confirmation prompts, changed button label to 'Allow', r=fluent-reviewers,flod,mhughes
Reporter | ||
Comment 3•1 year ago
|
||
Comment on attachment 9364779 [details]
Bug 1865883 - added confirmation prompts, changed button label to 'Allow', r=mhughes
Beta/Release Uplift Approval Request
- User impact if declined: FX being set as the default mail-to handler feature will be half-baked.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: @gasofie worked on the QA plan for this feature as a whole, and @atrif has been testing it. This particular change is to make the text in the notification bar match the specs, as they expect.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The feature is behind a preference and nimbus experiment. The changes being submitted via this patch will only affect things when the preference and the nimbus experiment are enabled.
- String changes made/needed: Yes. Fluent file is in the diff. @flod is aware.
- Is Android affected?: No
Attachment #9364779 -
Flags: approval-mozilla-beta?
Reporter | ||
Updated•1 year ago
|
Flags: qe-verify+
Comment 4•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox122:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
Comment 5•1 year ago
•
|
||
Comment on attachment 9364779 [details]
Bug 1865883 - added confirmation prompts, changed button label to 'Allow', r=mhughes
Approved for 121.0b3
Attachment #9364779 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•1 year ago
|
status-firefox121:
--- → fixed
Updated•1 year ago
|
QA Whiteboard: [qa-triaged]
Comment 7•1 year ago
|
||
Verified as fixed on Windows 10x64 and Windows 11x64 with Firefox 121.0b3 and Firefox Nightly 122.0a1.
Updated•1 year ago
|
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•