document system add-on install and upgrade process

RESOLVED FIXED in Firefox 49

Status

()

Toolkit
Add-ons Manager
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: rhelmer, Assigned: rhelmer)

Tracking

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
We have moved a number of Firefox features to system add-ons (Pocket, Hello, and some less user-visible features like e10s rollout) and using Balrog to push out-of-release-cycle updates to these out as part of the "go faster" project.

While there was a PRD written before any work started, there are some edge cases that cause confusion about exactly how this process works, so Osmose took some time to document the current situation in more detail.

Let's land this in toolkit/docs/ and hook it up so "./mach docs" works and it is published to gecko.readthedocs.org
(Assignee)

Comment 1

2 years ago
(In reply to Robert Helmer [:rhelmer] from comment #0)
> Let's land this in toolkit/docs/ and hook it up so "./mach docs" works and
> it is published to gecko.readthedocs.org

After looking at how other toolkit/ code is doing this, I decided to put it in ./toolkit/mozapps/extensions/docs/ and make it Add-on Manager specific instead.
(Assignee)

Comment 2

2 years ago
Created attachment 8759736 [details]
Bug 1277898 - start documenting add-on manager code, starting with system add-on spec

Review commit: https://reviewboard.mozilla.org/r/57622/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/57622/
Attachment #8759736 - Flags: review?(gps)
(Assignee)

Updated

2 years ago
Attachment #8759736 - Flags: feedback?(mkelly)
(Assignee)

Comment 3

2 years ago
gps, do we need to do anything special to get this published on gecko.readthedocs.org or is this patch enough? Thanks!
Status: NEW → ASSIGNED
Flags: needinfo?(gps)
Comment on attachment 8759736 [details]
Bug 1277898 - start documenting add-on manager code, starting with system add-on spec

Got it built locally, looks good to me. Thanks rhelmer!
Attachment #8759736 - Flags: feedback?(mkelly) → feedback+
(Assignee)

Comment 5

2 years ago
Comment on attachment 8759736 [details]
Bug 1277898 - start documenting add-on manager code, starting with system add-on spec

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/57622/diff/1-2/
Attachment #8759736 - Flags: feedback+
(Assignee)

Comment 6

2 years ago
We were missing a code block around one of the example XML responses, that's fixed now.

Comment 7

2 years ago
Comment on attachment 8759736 [details]
Bug 1277898 - start documenting add-on manager code, starting with system add-on spec

https://reviewboard.mozilla.org/r/57622/#review54584

I didn't really look at the content but the Sphinx integration looks correct.
Attachment #8759736 - Flags: review?(gps) → review+

Comment 8

2 years ago
gecko.readthedocs.io updates from mozilla-central every few hours. Just land this and it should appear.
Flags: needinfo?(gps)

Comment 9

2 years ago
Pushed by rhelmer@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9d34386dec39
start documenting add-on manager code, starting with system add-on spec r=gps

Comment 10

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9d34386dec39
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.