-
Notifications
You must be signed in to change notification settings - Fork 72
Undefined method find_asset
#97
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
Comments
Same problem here, no update but error message : HTML :
Error :
|
Same problem. It looks like adding Webpacker support may have implicitly broken the gem for people who do not have webpacker installed. |
@guillaumewrobel thanks for opening this issue and sorry about the error. Could you tell me:
You mentioned that you're seeing this in production. Is it reproducible in development also? (P.S. anybody else on this thread experiencing this problem, please feel free to also answer these questions)
When Webpacker is not installed the gem should try to use Sprockets by default and fallback to the naive |
The same error for me too. |
Thanks for the information, @bgmat. I was able to reproduce this on a test Heroku app. I've shipped a fixed version that falls back to the static asset finder (4c0da9e) in v1.5.1. Please give it a try and let me know if it doesn't fix the problem for you. Sorry for the trouble, everybody. The gem has decent test coverage, but the failure in this case was (of course) in the one untested part: The Railtie. This has remained largely untested because it's kind of painful to test a Railtie in a gem (all the Rails dependencies). I've started looking into lightweight ways to unit test Railties without requiring all of Rails so I can hopefully prevent this class of problem in future. |
same config as @bgmat Rails 5.2.3 on Heroku |
@guillaumewrobel thanks for the info. Please try |
|
Sorry for late response No webpack I'll give a try this week to |
Hello,
Just upgraded gem from 1.4.0 to 1.5.0 and this error came in production. Nothing changed on my side:
html
error
Probably linked to the webpack code.
Happy to help if you need more info.
The text was updated successfully, but these errors were encountered: