Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make _loadFromServer respect the base_url setting #112

Merged
merged 1 commit into from
May 9, 2019

Conversation

tubaman
Copy link
Contributor

@tubaman tubaman commented Apr 27, 2019

Make _loadFromServer respect the base_url setting. That way when we
serve wq from a non-root(/) path, the routing works correctly. For
example, try.wq.io "View All Observations" 404s without this.

@tubaman tubaman force-pushed the loadFromServer_base_url branch from dadaa21 to 470eb7e Compare April 27, 2019 17:02
@tubaman
Copy link
Contributor Author

tubaman commented Apr 27, 2019

I think the CI failure above has to due with a race condition with the phantomjs-based testing. I had similiar issues when running the tests locally.

@sheppard
Copy link
Member

sheppard commented May 8, 2019

Thanks, this would be good to have.

I integrated a newer testing library (jest) as part of #109. Could you rebase this PR off of the latest master?

Make _loadFromServer respect the base_url setting.  That way when we
serve wq from a non-root(/) path, the routing works correctly.  For
example, try.wq.io "View All Observations" 404s without this.
@sheppard sheppard force-pushed the loadFromServer_base_url branch from 470eb7e to 0e49b1e Compare May 9, 2019 01:01
@sheppard sheppard merged commit 0e49b1e into wq:master May 9, 2019
@sheppard
Copy link
Member

sheppard commented May 9, 2019

Never mind, I was able to rebase and push to your branch. Thanks!

@sheppard sheppard added this to the 1.1.2 - NPM Integration milestone May 21, 2019
@tubaman tubaman deleted the loadFromServer_base_url branch August 21, 2019 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants