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

Downgrade wkhtmltopdf #30

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcos-mendez
Copy link

The latest wkhtml version provided by Debian is incompatible with odoo and probably won't be maintained anymore, the reason is that Odoo os changing the ORM for other ways to manipulated PDFs. Te monkeypatch i found is to use the specific use of this: https://github.com/odoo/odoo/wiki/Wkhtmltopdf specially for Odoo16.0 https://github.com/wkhtmltopdf/packaging/releases/tag/0.12.6.1-3 The problem here is that it is CPU oriented. The main usage here for me has been AMD64, so probably you should also check this for your self

Making this a PR so people can see the main issue, naturally is not a TKL issue but rather than an Odoo Issue, any way im leaving this here for you to be able to check and don't have the need to deep dive in Odoo Documentation, also notices i'll leave this as Draft as this is not an actual FIX but rather dummy code. After release of Version 19.0 i remove this

The latest wkhtml version provided by Debian is incompatible with odoo and probably won't be maintained anymore, the reason is that Odoo os changing the ORM for other ways to manipulated PDFs. Te monkeypatch i found is to use the specific use of this: https://github.com/odoo/odoo/wiki/Wkhtmltopdf specially for Odoo16.0 https://github.com/wkhtmltopdf/packaging/releases/tag/0.12.6.1-3
The problem here is that it is CPU oriented. The main usage here for me has been AMD64, so probably you should also check this for your self
@marcos-mendez marcos-mendez marked this pull request as draft January 7, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant