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

[FW][FIX] tools: prevent IoT crash due to missing library #165150

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented May 10, 2024

Before this commit:
Any IoT-box syncing to a version 16 (and >) database would not be able to start odoo service due to the num2words missing library.

This is in practice the fault of the IoT as this library is part of odoo requirement. But to solve it it would need to recreate an IoT box OS image and asking customers to re-flash their SD card which is not convenient and time consumming.

After this commit:
Import conditionally num2words

opw-3902183

Forward-Port-Of: #165067

Before this commit:
Any IoT-box syncing to a version 16 (and >) database would not be
able to start odoo service due to the `num2words` missing library.

This is in practice the fault of the IoT as this library is part
of odoo requirement. But to solve it it would need to recreate an
IoT box OS image and asking customers to re-flash their SD card
which is not convenient and time consumming.

After this commit:
Import conditionally num2words

opw-3902183

X-original-commit: 7d87721
@robodoo
Copy link
Contributor

robodoo commented May 10, 2024

@fw-bot
Copy link
Contributor Author

fw-bot commented May 10, 2024

This PR targets saas-17.2 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label May 10, 2024
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label May 10, 2024
robodoo pushed a commit that referenced this pull request May 11, 2024
Before this commit:
Any IoT-box syncing to a version 16 (and >) database would not be
able to start odoo service due to the `num2words` missing library.

This is in practice the fault of the IoT as this library is part
of odoo requirement. But to solve it it would need to recreate an
IoT box OS image and asking customers to re-flash their SD card
which is not convenient and time consumming.

After this commit:
Import conditionally num2words

opw-3902183

closes #165150

X-original-commit: 7d87721
Signed-off-by: Pierre Masereel (pim) <pim@odoo.com>
Signed-off-by: Loan Sens (lse) <lse@odoo.com>
@robodoo robodoo closed this May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants