Closed
Bug 614339
Opened 14 years ago
Closed 14 years ago
move documents for SDK developers to Internals section
Categories
(Add-on SDK Graveyard :: Documentation, defect)
Add-on SDK Graveyard
Documentation
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b1
People
(Reporter: myk, Assigned: wbamberg)
References
Details
Attachments
(1 file)
19.07 KB,
patch
|
avarma
:
review+
|
Details | Diff | Splinter Review |
Documents that are intended for the SDK's own developers (i.e. folks hacking on the SDK itself) should be moved to an Internals section of the documentation to make it clear that they are not intended for a general addon developer audience.
The documents to move are:
Low-Level Module Best Practices
Chrome Authority/Manifest Generation
XPI Generation
the __url__, packaging, and memory explanations in Reference > Globals
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → wbamberg
Assignee | ||
Comment 1•14 years ago
|
||
Note that this patch is supposed to address both 614339 and 614340, since they're quite intertwined. I've created new "Experimental" and "Internals" sections and reorganized them as this defect asks. I've split off some of the globals into a new "Internal Globals" page. I've also split out "Program ID" into its own page so it's easy to reference it from other places, because both the tutorial and the XPI Generation page want to reference it.
I also noticed and fixed what seems to be a bug in "Programs", where cfx actually generates the program ID when you do "cfx run" not "cfx xpi".
This change can also be found at https://github.com/wbamberg/jetpack-sdk/tree/bug614339 if that makes your life any easier.
Attachment #493022 -
Flags: review?(avarma)
Comment 3•14 years ago
|
||
Looks great, I've pushed it here:
https://github.com/mozilla/addon-sdk/commit/28013e9ca3ba66e548ceea84d4426e4f1c87eca6
However, I noticed some errors in it that are either minor or not actually relevant to this bug, and commented on them in a separate branch on my repo:
https://github.com/toolness/jetpack-sdk/commit/28013e9ca3ba66e548ceea84d4426e4f1c87eca6
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Attachment #493022 -
Flags: review?(avarma) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•