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

Provide the base_path for asset lookup #688

Merged
merged 1 commit into from
Oct 3, 2017

Conversation

PragTob
Copy link
Contributor

@PragTob PragTob commented Oct 2, 2017

First, thanks for wicked_pdf! 💚

Ran into this problem while upgrading sprockets on our application
a rather lengthy debugging session through sprockets and friends
ended up here showing that the asset could be found if the
base_path was supplied. Iirc it's optional and should only
help - tbh it's a bit ago so don't have all the code in my
head anymore :)

edit: also we are running this in production so at least for us it works (tm) with the newest sprockets

Ran into this problem while upgrading sprockets on our application
a rather lengthy debugging session through sprockets and friends
ended up here showing that the asset could be found if the
base_path was supplied. Iirc it's optional and should only
help - tbh it's a bit ago so don't have all the code in my
head anymore :)
@unixmonkey unixmonkey merged commit 537fc2d into mileszs:master Oct 3, 2017
@unixmonkey
Copy link
Collaborator

Thanks!

@PragTob PragTob deleted the asset-base-path branch October 4, 2017 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants