Gem version should be bumped for wicked_pdf support #4

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
@camdez
Contributor

camdez commented Mar 23, 2012

Gem version should be bumped so that wicked_pdf users have a minimum version they can count on for wicked_pdf support.

Apologies on not thinking of this before.

@nat

This comment has been minimized.

Show comment Hide comment
@nat

nat Apr 19, 2012

Hi I want to write and support this pull request of camdez'. His fix addresses a particular issue that I've been having as well. Thank you!

nat commented Apr 19, 2012

Hi I want to write and support this pull request of camdez'. His fix addresses a particular issue that I've been having as well. Thank you!

@nat

This comment has been minimized.

Show comment Hide comment
@nat

nat Apr 19, 2012

Also you'll see more of the rationale for this request here:
camdez#1

Thanks!

nat commented Apr 19, 2012

Also you'll see more of the rationale for this request here:
camdez#1

Thanks!

@nat

This comment has been minimized.

Show comment Hide comment
@nat

nat Apr 24, 2012

As further clarification. When I am using Heroku cedar with my particular heroku app, when I use the gem you've released it breaks the load and I cannot load any of my heroku instances where it's included. And Camdez' fix works for me perfectly (it's what I'm using), but yours is superior because it's a gem and just 11MB compared to Camdez' which is 34 MB because for some reason when I use gem "wkhtmltopdf-heroku", :git => 'git://github.com/camdez/wkhtmltopdf-heroku.git' in bundler, heroku includes the .git repository.

I've corresponded with Heroku support and pointed them to our discussions and their exact response/fix was:
"Hopefully opening up these issues will encourage the maintainers to update the gem releases."

Thanks for your great gem. I'm sure you are busy and hope you can do this!
Sincerely,
Nathan wharton

nat commented Apr 24, 2012

As further clarification. When I am using Heroku cedar with my particular heroku app, when I use the gem you've released it breaks the load and I cannot load any of my heroku instances where it's included. And Camdez' fix works for me perfectly (it's what I'm using), but yours is superior because it's a gem and just 11MB compared to Camdez' which is 34 MB because for some reason when I use gem "wkhtmltopdf-heroku", :git => 'git://github.com/camdez/wkhtmltopdf-heroku.git' in bundler, heroku includes the .git repository.

I've corresponded with Heroku support and pointed them to our discussions and their exact response/fix was:
"Hopefully opening up these issues will encourage the maintainers to update the gem releases."

Thanks for your great gem. I'm sure you are busy and hope you can do this!
Sincerely,
Nathan wharton

@bradphelan

This comment has been minimized.

Show comment Hide comment
@bradphelan

bradphelan May 27, 2012

Collaborator

I've bumped the version to 1.0 but it's pusing to rubygems.org at the moment. Hopefully it completes.

Collaborator

bradphelan commented May 27, 2012

I've bumped the version to 1.0 but it's pusing to rubygems.org at the moment. Hopefully it completes.

@nat

This comment has been minimized.

Show comment Hide comment
@nat

nat May 31, 2012

Brad, I think I just attempted to push your update to Heroku but it seemed to crash my application, still. I haven't had time to debug it fully.. not sure if it's just because rubygems wasn't updated or not. Thanks!

nat commented May 31, 2012

Brad, I think I just attempted to push your update to Heroku but it seemed to crash my application, still. I haven't had time to debug it fully.. not sure if it's just because rubygems wasn't updated or not. Thanks!

@cdwort

This comment has been minimized.

Show comment Hide comment
@cdwort

cdwort Aug 27, 2013

@camdez @nat Some of the issues with wkhtmltopdf-heroku have been affecting PDFKit, so we've forked this repo under the PDFKit org. If this is still an problem for you, please open an issue at https://github.com/pdfkit/wkhtmltopdf-heroku/

Thanks!

cdwort commented Aug 27, 2013

@camdez @nat Some of the issues with wkhtmltopdf-heroku have been affecting PDFKit, so we've forked this repo under the PDFKit org. If this is still an problem for you, please open an issue at https://github.com/pdfkit/wkhtmltopdf-heroku/

Thanks!

@pallymore pallymore closed this Mar 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment