Closed Bug 1331521 Opened 5 years ago Closed 4 years ago

Remove about:newaddon when new sideloading flow is in place

Categories

(Toolkit :: Add-ons Manager, defect, P2)

51 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: aswan, Assigned: aswan)

References

Details

(Whiteboard: triaged)

Attachments

(1 file)

Once bug 1317363 is done, we'll have a new flow in place for sideloaded add-ons.  There are a few other things that need to be done before we turn on the new flow by default (tracked in bug 1308292) but once that is done, the old code for about:newaddon should be removed.
Depends on: webext-permissions
Priority: -- → P2
Whiteboard: triaged
Duplicate of this bug: 1392770
(In reply to Andrew Swan [:aswan] from comment #0)
> Once bug 1317363 is done, we'll have a new flow in place for sideloaded
> add-ons.  There are a few other things that need to be done before we turn
> on the new flow by default (tracked in bug 1308292) but once that is done,
> the old code for about:newaddon should be removed.

All of these bugs are closed - can we remove this now? :-)
Flags: needinfo?(aswan)
Yep, I believe this is good to go as soon as somebody has the time to do it.
Flags: needinfo?(aswan)
Assignee: nobody → aswan
Attachment #8968368 - Flags: review?(kmaglione+bmo)
Comment on attachment 8968368 [details]
Bug 1331521 Remove about:newaddon

https://reviewboard.mozilla.org/r/237052/#review242860

\o/
Attachment #8968368 - Flags: review?(kmaglione+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/c85fefa9248c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Is manual testing required on this bug? If yes, please provide some STR and the proper extension(if required) or set the “qe-verify -“ flag.

Thanks!
Flags: needinfo?(aswan)
Flags: needinfo?(aswan) → qe-verify-
You need to log in before you can comment on or make changes to this bug.