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

I18n js fixes for lodash changes. #5

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

Conversation

JDutil
Copy link

@JDutil JDutil commented Apr 12, 2017

When using the latest i18n-js and webpacker 2 these changes are required for compatibility.

See also #2 & #3 which this resolves as well using #2 solution. Would be better to merge #2 before this PR for their credit I was too lazy to cherry-pick it.

@suusan2go
Copy link

@JDutil Hi, I had the same lodash reference problem. (fixed at this commit 7d9b5c7)

I think this is webpacker config issue ( Actually, webpack config. )
See this PR.
rails/webpacker#166

@JDutil
Copy link
Author

JDutil commented Apr 13, 2017

@suzan2go interesting but I just tried that solution, and I still needed to apply these lodash changes to fix my compilation errors for now.

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.

2 participants