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

Prepare for FastBoot 1.0 #27

Closed
wants to merge 1 commit into from
Closed

Conversation

kratiahuja
Copy link

@kratiahuja kratiahuja commented Jun 27, 2017

Fixes #26

PS: I haven't tested this, so please test it in an app.

cc: @jkusa

@jkusa
Copy link
Owner

jkusa commented Jun 29, 2017

@kratiahuja a quick test with your change, ember-2.13.3, and fastboot-1.0.0-rc.5 did not work in the dummy app. I'll see if I can dig in a little more.

@sivakumar-kailasam
Copy link
Contributor

@kratiahuja @jkusa I've tried changes on an actual app and raised #28

@kratiahuja
Copy link
Author

Curious to understand what didn't work?

@jkusa
Copy link
Owner

jkusa commented Jul 7, 2017

@kratiahuja thanks for getting the ball rolling with this PR. #28 is now merged and includes these changes along with additional logic to check for the presence of window.

@jkusa jkusa closed this Jul 7, 2017
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.

None yet

3 participants