Closed
Bug 658043
Opened 14 years ago
Closed 14 years ago
Change /api to redirect to jetpack.mozillalabs documentation
Categories
(addons.mozilla.org Graveyard :: Add-on Builder, defect, P1)
addons.mozilla.org Graveyard
Add-on Builder
Tracking
(Not tracked)
RESOLVED
FIXED
Builder 0.9.5
People
(Reporter: smcarthur, Assigned: smcarthur)
References
Details
We will no longer host the documentation, removing the burden of having up to date docs. We should continue to allow the /api link, only make it a 302 to whichever is the URL for the docs, whether its jetpack.mozillalabs.com, or MDN.
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → smcarthur
Priority: -- → P1
Assignee | ||
Updated•14 years ago
|
Target Milestone: --- → Builder 0.9.5
Assignee | ||
Comment 1•14 years ago
|
||
The latest URL on the jetpack site does a blanket redirect to the the root of the docs, and drops any extra part of the path, so to allow deeplinking of the docks, we would need to update our settings to the real latest url. However, this doesn't seem like much of an issue since we currently need to do a push with the new SDK version for every new version anyways.
@zalun: Also, our schema's a slightly different. In order to use generic redirect_to views, we would need to split api_page into api_package_page and api_module_page. Otherwise, we could skip using a generic, and make a single view split the url and handle the logic ourselves. I'm open to both.
FlightDeck | Jetpack
/addon-kit | /addon-kit/addon-kit.html
/addon-kit/panel | /addon-kit/docs/panel.html
Assignee | ||
Comment 2•14 years ago
|
||
We'll just do this with Apache. However, it will also depend on Jetpack getting their rewrite to allow deeper linking.
Assignee | ||
Comment 3•14 years ago
|
||
Do we do this with a .htaccess file, or provide the lines needed for the apache conf, or something else?
Comment 4•14 years ago
|
||
either is fine (they're pretty much the same). If we don't have a .htaccess already, it's easy to stick it in the vhost, we just need the regex
Assignee | ||
Comment 5•14 years ago
|
||
in master https://github.com/seanmonstar/FlightDeck/commit/b8069c16a02de80dfe0f8f2044fa86d09b61941d
Will need in vhosts:
/docs/(.*) https://jetpack.mozillalabs.com/sdk/latest/docs/$1
Assignee | ||
Comment 6•14 years ago
|
||
With 661073 completed, this is done.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•