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

[IMP] client to return mocked server version #8

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

Conversation

nilshamerlinck
Copy link
Contributor

Odoo checks for wkhtmltopdf's version on start.

In a CI context, we don't necessarily want to run the kwkhtmtopdf container.

Provided with KWKHTMLTOPDF_SERVER_URL=MOCK environment variable, the client will return "wkhtmltopdf 0.12.5 (mock)" and Odoo will start properly.

@sbidoul
Copy link
Member

sbidoul commented Apr 26, 2019

Sounds legit. Can you update the documentation and add a small test for that?

@sbidoul sbidoul added the enhancement New feature or request label Apr 26, 2019
@nilshamerlinck
Copy link
Contributor Author

Sure, will do.

@nilshamerlinck
Copy link
Contributor Author

Done @sbidoul

@sbidoul
Copy link
Member

sbidoul commented Apr 10, 2022

@nilshamerlinck I had definitely lost track of this. If you still need this, can you rebase as I've done some CI maintenance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants