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

Add note to avoid confusion when following the book #293

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

coelhucas
Copy link

@coelhucas coelhucas commented Apr 16, 2023

Summary

While following the book I had this small issue due to newer versions of Node.js not working on older Webpack.

Instead of reworking the template with a newer Webpack version I decided to not only add how to solve, but also a quick overview that might interest other seasoned developers reading.

Post due diligence acknowledgments (edit):
Fixes #290, #279 and #273.

As pointed by @nik0sc here, another possible fix would be rustwasm/create-wasm-app#207, but since many people use older webpack versions & due to the amount of change needed in the other solution, I think this is enough for learning purposes.

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.

[4.2] Serving Locally fails
1 participant