Victor Champonnois
Victor Champonnois
@legalsylvain The travis CI is red because of a timeout. Do you have an idea on how to fix this ?
Note to @carmenbianca : I did not explicitly resolve the promise in the code. - I am not sure it is needed, since the screen is locked until the mail...
> @victor-champonnois : not sure that is a time out problem. when executing test of `pos_mail_receipt` something seems blocked. > > `No output has been received in the last 10m0s`......
@legalsylvain This is tricky. The timeout happens when the test call `send_mail_receipt()`, [itself calling `_run_wkhtmltopdf`](https://github.com/OCA/pos/blob/df17842e7cdce02eae825c1133c905dc2ae884f5/pos_mail_receipt/models/pos_order.py#L52) to convert the receipt to a pdf. `_run_wkhtmltopdf` run wkhtmltopdf in a subprocess. I tried...
@ivantodorovich Thanks a lot, it works :)
It seems that the link between task and milestone is standard in Odoo 16. So `project_task_milestone` does not need to be ported. `project_milestone` is standard since 15.0, so I think...
@robinkeunen use case added in the commit message
@robinkeunen unit test added and description updated.
@robinkeunen fixed the test (as in https://github.com/OCA/account-analytic/pull/622), but not precommit (it's not setup in this repo)