Active Storage Isn't Playing Nice with Prawn-Rails

[seanmarcia]

Since we switched to active storage our pdf generation on distributions is not working correctly.

We attempt to pull in the logo of the diaper banks in the pdfs we generate app/views/distributions/print.pdf.prawn on lines 2 and 120. Using the logo_file_path which is a helper method but something with the recent active storage has broken this.

This you can see the helper method in the following file: app/helpers/distributions_helper.rb

We need to get the images properly inserted back into the generated pdfs.

This is hopefully a simple fix.

To reproduce you need to upload a diaper bank logo so the file prawn is pulling in is one that is being accessed through active storage.

I've attached a logo from one of our partner diaper banks for testings.

[davidsth]

@seanmarcia Maybe I'm missing something. I cannot reproduce it locally. The logo is overlapping the text on the right and that can be fixed.

[seanmarcia]

@davidshr10 I updated the instructions to reproduce!

You are correct that the default image is way too wide. We should use the fit attribute with the logo.

Something like: pdf.image logo_file_path(current_organization), fit: [325, 110]

[seanmarcia]

Re-opening as this breaks after a deploy. /cc @davidshr10

[cattywampus]

As discussed in the 9/15 Hack day, looks like each new capistrano deploy loses the storage directory where the uploads are placed. This needs to be moved into the capistrano shared directory and relinked to the new release when deployed. I can tackle this.

[cattywampus]

I think this is resolved by Pull Request #475