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

Astronomy not working with webpack #688

Closed
Deadly0 opened this issue Sep 28, 2018 · 7 comments
Closed

Astronomy not working with webpack #688

Deadly0 opened this issue Sep 28, 2018 · 7 comments
Labels

Comments

@Deadly0
Copy link

Deadly0 commented Sep 28, 2018

ardatan:webpack provides a way to use Webpack with meteor. But for some reasons Astronomy can't be imported in project by webpack. I have reported this bug to @ardatan but he said it's not meteor-webpack bug ardatan/meteor-webpack#40

It's important to me to migrate to Webpack, but i can't do it becouse of this bug with Astronomy :(

Thank you!

@Deadly0 Deadly0 changed the title astronomy not working with webpack Astronomy not working with webpack Sep 28, 2018
@lukejagodzinski
Copy link
Member

It's definitely not something related with Astronomy. I would rather say that's it's a bug in webpack package. Astronomy is working normally when you just use Meteor alone. In the last version of Astronomy I've introduced lazy loading, so maybe this webpack plugin does not work well with this kind of scenarios. Try using it with different Astronomy versions and see if it's producing the same error. And if it's not gonna help then create a reproduction repository so I will be able to check it

@Deadly0
Copy link
Author

Deadly0 commented Sep 28, 2018

@lukejagodzinski you are right, old versions works fine. In which version lazy loading was introduced?

@lukejagodzinski
Copy link
Member

@Deadly0 it was the last one version 2.6.0. In the version 2.5.8 there was no lazy loading.
So as I said it's maybe not bug but a lack of feature in the webpack package. It's not so uncommon to have Meteor packages with the lazy loading feature, so I think that the webpack package should take that into account.

@Deadly0
Copy link
Author

Deadly0 commented Sep 28, 2018

@lukejagodzinski thank you!

@Deadly0 Deadly0 closed this as completed Sep 28, 2018
@alexpricedev
Copy link

Yeah I'm having the same issue. @Deadly0 did you resolve it?

@Deadly0
Copy link
Author

Deadly0 commented Oct 12, 2018

@Deadly0 no, but workaround is to use v2.5.8 of astronomy.

@alexpricedev
Copy link

Yeah thats that I've done! Let me know if you look into it again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants