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

When using the liveReloadBaseUrl with an absolute URL, the non-path portion should be stripped when setting up middleware. #36

Merged
merged 1 commit into from
Jul 14, 2017

Conversation

lukemelia
Copy link
Contributor

No description provided.

@machty
Copy link

machty commented Oct 28, 2016

Plus. One.

@kevinwmerritt
Copy link

Any chance this will get merged?

…ortion should be stripped when setting up middleware.
@jbescoyez
Copy link
Contributor

This PR fixes a regression mentioned in #38. The only workaround I found so far is to use an old version of ember-cli-inject-live-reload. It would be great to see this merged. /ping @rwjblue

@rwjblue rwjblue merged commit 9d9749f into ember-cli:master Jul 14, 2017
@rwjblue
Copy link
Member

rwjblue commented Jul 14, 2017

Very sorry for the long delay here. Sorry for letting this fall through the cracks.

@lukemelia
Copy link
Contributor Author

Better late than never @rwjblue! Thanks! 🍻

@jbescoyez
Copy link
Contributor

Yeah! Thanks @rwjblue.

@rwjblue rwjblue added the bug label Jul 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants