Closed Bug 1277898 Opened 4 years ago Closed 4 years ago

document system add-on install and upgrade process

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: rhelmer, Assigned: rhelmer)

Details

Attachments

(1 file)

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
(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.
Attachment #8759736 - Flags: feedback?(mkelly)
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+
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+
We were missing a code block around one of the example XML responses, that's fixed now.
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+
gecko.readthedocs.io updates from mozilla-central every few hours. Just land this and it should appear.
Flags: needinfo?(gps)
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
https://hg.mozilla.org/mozilla-central/rev/9d34386dec39
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.