Closed
Bug 769290
Opened 12 years ago
Closed 12 years ago
Document testing the client-side of OrangeFactor from the local filesystem
Categories
(Tree Management Graveyard :: OrangeFactor, defect)
Tree Management Graveyard
OrangeFactor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
Details
Attachments
(1 file)
1.80 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
TBPL allows you to: 1) Check out the repo 2) Change a baseURL pref in Config.js to point to the production backend at tbpl.m.o 3) Run index.html from the local filesystem -> Which lets you test client-side changes, if you don't have the TBPL server component set up. It would be great if we could do something similar for OrangeFactor, since at present it just gets stuck on "Loading..."
Comment 1•12 years ago
|
||
In fact you can do this already, following the steps you outlined above, replacing step 2 with 2) Change the definition of the "es_server" variable in html/index.html (currently on line 164) to "http://brasstacks.mozilla.com/orangefactor/api" We already have Access-Control-Allow-Origin set to * so there shouldn't be any AJAX cross-domain restrictions.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Updated•12 years ago
|
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Summary: Make it possible to test the client side of OrangeFactor from the local filesystem → Document testing the client-side of OrangeFactor from the local filesystem
Comment 3•12 years ago
|
||
Comment on attachment 637570 [details] [diff] [review] Add instructions Review of attachment 637570 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, glad to see someone else working on OF! Just a couple grammar fixes. :) ::: README.txt @@ +8,5 @@ > > (TODO: Replace all this with a script...) > + > + > +To test just the client-side: Technically this should be "client side" since you are using "side" as a noun. ;) @@ +11,5 @@ > + > +To test just the client-side: > + > +* checkout this code locally > +* change the es_server variable in html/index.html to: "http://brasstacks.mozilla.com/orangefactor/api" Also technically no ":".
Attachment #637570 -
Flags: review?(mcote) → review+
Assignee | ||
Comment 4•12 years ago
|
||
With the corrections: http://hg.mozilla.org/automation/orangefactor/rev/4215967d3ae7 (Btw I <3 the attention to detail, please keep it coming in future reviews \o/)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Testing → Tree Management
Updated•4 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•