Sorry I really just wanted to push a PR to my own fork. That said, I patched Attachment#diskfile to return the proper S3 url and Attachment#readable? to return true if the file exists on S3.
Then, and that relates to PDF rendering, I patched the PDF renderer to use open_uri, for the former implementation was not able to get our images porperly.
Sorry I really just wanted to push a PR to my own fork. That said, I patched Attachment#diskfile to return the proper S3 url and Attachment#readable? to return true if the file exists on S3.
Then, and that relates to PDF rendering, I patched the PDF renderer to use open_uri, for the former implementation was not able to get our images porperly.