-
Notifications
You must be signed in to change notification settings - Fork 63
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
Version 0.7.2 breaks Glimmer.js app #146
Comments
@marten-dockyard could someone give me feedback on this issue? If we want to support glimmer.js and if we have a rough idea how to do so, I could try to create a PR 🙂 |
would be great to get some opinion about this problem. As I said, I could maybe help to make this addon available for glimmer.js |
Looks like this would have started here: 9d6db8a#diff-61e7a77a0dda836b877cba3567f38581 |
@eshtadc thanks for the response :-) if I add |
We used
ember-service-worker
with our Glimmer.js app and it just worked fine with versionv0.6.2
now we tried to upgrade to the newest version which is currentlyv0.7.2
but this upgrade broke our build. This is due to the fact, that Glimmer.js has not installedember-cli-babel
and therefore the following code breaks:ember-service-worker/lib/service-worker-builder.js
Line 125 in 5ffad24
On the other hand adding
ember-cli-babel
breaks our Glimmer.js build. So I'm not sure how to resolve this issue.I know that Ember.js is not Glimmer.js and therefore I'm not sure if this is an issue with
ember-service-worker
or a Glimmer.js issue. Nevertheless it would be great to share as much addons and plugins as possible between Ember.js and Glimmer.jsJust let me know what you think about this issue
The text was updated successfully, but these errors were encountered: