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

[17.0] delivery: neutralize should only disable production environment and not disable carriers #164519

Open
john-herholz-dt opened this issue May 6, 2024 · 1 comment
Labels
Logistics about stock, mrp, delivery, purchase

Comments

@john-herholz-dt
Copy link

Shipping providers have a test and production environment, to process API calls.
When neutralizing a database, it should switch to the test environment of the providers.

Impacted versions:
17.0

Steps to reproduce:
Neutralize a database

Current behavior:
After database neutralization, all shipping carriers are disabled.
For test environments, this means you need to manually enable all carriers and leave it in development environment.

Expected behavior:
Only production environment is disabled on neutralization

Video/Screenshot link (optional):

Support ticket number submitted via odoo.com/help (optional):
[FW][IMP] delivery: disable only delivery carriers from external providers #161182
Was already a move in the right direction, but the issue here is still present.

@vava-odoo
Copy link
Contributor

Hi @john-herholz-dt
Thanks for your feedback. Actually, I came up with a similar idea (see discussion in #160443) but @ticodoo thought it was safer to implement an intermediate version in stable.
I wonder if it couldn't be as you mention in master, with delivery only having

UPDATE delivery_carrier
   SET prod_environment = false;

and maybe a dedicated neutralize script for external providers that require further actions to ensure safe usage.

@vava-odoo vava-odoo added the Logistics about stock, mrp, delivery, purchase label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Logistics about stock, mrp, delivery, purchase
Projects
None yet
Development

No branches or pull requests

2 participants