Closed
Bug 799950
Opened 12 years ago
Closed 12 years ago
OrangeFactor should adjust 'es_server' automatically if testing the client-side 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.56 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
This is OrangeFactor's equivalent of bug 799908. Currently we can test client-side only changes by running index.html from the local filesystem & pointing the UI at the production server-side components. However this requires changing es_server in woo.config.js every time I want to test locally. I do this by popping/pushing an mq, but it's still a PITA to have to remember to do - and if I forget to push the mq before starting the next patch, I inevitably end up at some point qrefreshing parts of the patch into my baseURL mq patch. We should just detect "file:" in window.location.protocol and set the value automatically to save this messing about.
Assignee | ||
Comment 1•12 years ago
|
||
Happy to adjust the wording (or line wrapping of the ternary) if needed :-)
Attachment #669971 -
Flags: review?(mcote)
Comment 2•12 years ago
|
||
Comment on attachment 669971 [details] [diff] [review] Patch v1 Review of attachment 669971 [details] [diff] [review]: ----------------------------------------------------------------- Very cool! I suppose eventually we'll want to change this to point to our dev server when it's up, but good for now.
Attachment #669971 -
Flags: review?(mcote) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Cheers :-) https://hg.mozilla.org/automation/orangefactor/rev/b408e761bd66
Status: ASSIGNED → RESOLVED
Closed: 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
•