-
Notifications
You must be signed in to change notification settings - Fork 52
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 fastboot guard #26
Conversation
Thanks for this! Could you help me understand how to verify the issue so I can understand the fix? I've been thinking about fastboot lately, so this is a great way to get in :) |
When I started my app with
This basically means that nouislider.js uses some functionality which is not available on server side rendering. My fix just excludes all libraries which are not available in fastboot mode. Maybe you could write some polyfills but for our app it was/is sufficient to just exclude things which do not work on the server. Does this make sense? |
Makes perfect sense! Will have to see how it can be added to the tests as
Kenneth Kalmer |
@kennethkalmer I accidentally put the fastboot guard into the wrong file. With my latest commit I switched the guard to index.js. If you have any questions let me know |
Any update on this? |
Thanks for this and sorry for letting it sit. Will prepare a patch release shortly |
I just released 0.10.0, could you please let me know if there are any regressions? |
I implemented a simple fastboot guard so that the fastboot server can start and is not crashing while booting. I tried to implement a fixed base on the following discussions: ember-fastboot/ember-cli-fastboot#105 and plusacht/ember-bootstrap-datetimepicker#75