Closed Bug 769290 Opened 10 years ago Closed 10 years ago

Document testing the client-side of OrangeFactor from the local filesystem

Categories

(Tree Management Graveyard :: OrangeFactor, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

Details

Attachments

(1 file)

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..."
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: 10 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Attached patch Add instructionsSplinter Review
Assignee: nobody → bmo
Status: REOPENED → ASSIGNED
Attachment #637570 - Flags: review?(mcote)
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 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+
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: 10 years ago10 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.