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

Bunch -> Webpack #32

Merged
merged 1 commit into from
Nov 10, 2018
Merged

Bunch -> Webpack #32

merged 1 commit into from
Nov 10, 2018

Conversation

stevegrossi
Copy link
Owner

@stevegrossi stevegrossi commented Nov 9, 2018

I’m following https://gist.github.com/chrismccord/bb1f8b136f5a9e4abc0bfc07b832257e

Issues to Resolve

  • SCSS changes aren’t reflected in the browser
  • import css from '../css/app.css'; fails, as the link above does not get into using SCSS
  • ReferenceError: require is not defined in the console, but JS still works. I get another error, Error: Cannot find module 'web/static/js/app' from ‘/‘ on master, so this may be related to a pre-existing issue.

@stevegrossi stevegrossi force-pushed the stevegrossi/brunch-to-webpack branch 3 times, most recently from 118cba7 to 586d34d Compare November 10, 2018 11:34
I’m following https://gist.github.com/chrismccord/bb1f8b136f5a9e4abc0bfc07b832257e

~~Current~~ issues:

- SCSS changes aren’t reflected in the browser
- `ReferenceError: require is not defined` in the console, but JS still works. I get another error, `Error: Cannot find module 'web/static/js/app' from ‘/‘` on master, so this may be related to a pre-existing issue.

This helped me get SCSS working: https://elixirforum.com/t/phoenix-1-3-to-1-4-migrating-from-brunch-to-webpack/17820/4

And the `require()` script I had in application.html.eex doesn’t appear necessary and isn’t present in new phoenix 1.4 apps, so I’m not sure where it came from.
@stevegrossi stevegrossi merged commit 5a1f130 into master Nov 10, 2018
@stevegrossi stevegrossi deleted the stevegrossi/brunch-to-webpack branch November 10, 2018 13:11
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.

1 participant