loader documentation examples don't point to the correct URL

RESOLVED FIXED

Status

Add-on SDK
Documentation
P3
normal
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: KWierso, Assigned: jsantell)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The examples near the top of the page in the loader documentation uses resource://gre/modules/toolkit/loader.js as the URL for the loader, but that URL doesn't work for me when I try to use it.

resource://gre/modules/commonjs/toolkit/loader.js does work, though.



Filing this as a documentation bug to get the URL corrected.
(Reporter)

Comment 1

5 years ago
Irakli, is updating the docs the right thing to do, here, or should the URL currently in the docs work?
Flags: needinfo?(rFobic)
(Reporter)

Updated

5 years ago
Summary: loader documentation example doesn't point to the correct URL → loader documentation examples don't point to the correct URL
(Reporter)

Updated

5 years ago
Priority: -- → P3
Yes it should
Flags: needinfo?(rFobic)
No, this should not work -- that is the wrong URL.

The correct URL is:
resource://gre/modules/commonjs/toolkit/loader.js
(Assignee)

Updated

4 years ago
Assignee: nobody → jsantell
Created attachment 8345374 [details] [review]
GH PR 1321
Attachment #8345374 - Flags: review?(wbamberg)
Comment on attachment 8345374 [details] [review]
GH PR 1321

I think we should stop updating the docs on GitHub, and update them on MDN as soon as they are there. So let's keep this open and fix it on MDN as soon as the loader doc is there!
Attachment #8345374 - Flags: review?(wbamberg) → review-
Added to the new MDN docs! Closing this for now
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.