silverstripe / cwp-pdfexport

Adds PDF export capability for CWP pages
BSD 3-Clause "New" or "Revised" License
4 stars 6 forks source link

Update the generated_pdf_path #15

Closed MasseyIsaako closed 5 years ago

MasseyIsaako commented 5 years ago

I've been having issues with deploying to UAT for a recent project, turns out that the issue stems from the private static $generated_pdf_path, which references the SS3 asset path. I have changed this to public/assets/_generated_pdfs.

robbieaverill commented 5 years ago

This should have been fixed in #9, is that change not working for you?

MasseyIsaako commented 5 years ago

Hi @robbieaverill, I've been doing a few deployment recently and the fix from #9 actually does work. I'll close this pull request now. Thanks!