-
-
Notifications
You must be signed in to change notification settings - Fork 982
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][MIG] sale partner incoterm: Migration to 17.0 #2968
base: 17.0
Are you sure you want to change the base?
[17.0][MIG] sale partner incoterm: Migration to 17.0 #2968
Conversation
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-12.0/sale-workflow-12.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_partner_incoterm/
Currently translated at 100.0% (4 of 4 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_partner_incoterm/de/
Currently translated at 100.0% (4 of 4 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_partner_incoterm/zh_CN/
Currently translated at 100.0% (4 of 4 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_partner_incoterm/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-13.0/sale-workflow-13.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_partner_incoterm/
Currently translated at 100.0% (4 of 4 strings) Translation: sale-workflow-13.0/sale-workflow-13.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_partner_incoterm/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-13.0/sale-workflow-13.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_partner_incoterm/
Currently translated at 100.0% (7 of 7 strings) Translation: sale-workflow-14.0/sale-workflow-14.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_partner_incoterm/ca/
Currently translated at 75.0% (3 of 4 strings) Translation: sale-workflow-14.0/sale-workflow-14.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_partner_incoterm/nl/
Currently translated at 75.0% (3 of 4 strings) Translation: sale-workflow-14.0/sale-workflow-14.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_partner_incoterm/fr/
Currently translated at 100.0% (4 of 4 strings) Translation: sale-workflow-14.0/sale-workflow-14.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_partner_incoterm/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-15.0/sale-workflow-15.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_partner_incoterm/
A field for incoterm address is added in partner model, this works as incoterm field
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-16.0/sale-workflow-16.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_partner_incoterm/
Currently translated at 100.0% (6 of 6 strings) Translation: sale-workflow-16.0/sale-workflow-16.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_partner_incoterm/es/
Currently translated at 100.0% (6 of 6 strings) Translation: sale-workflow-16.0/sale-workflow-16.0-sale_partner_incoterm Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_partner_incoterm/it/
/ocabot migration sale_partner_incoterm |
1 similar comment
/ocabot migration sale_partner_incoterm |
) | ||
|
||
@api.onchange("partner_id") | ||
def onchange_partner_id_set_incoterm(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@StefanRijnhart As Odoo changed sale orders to full computed fields, shouldn't we use them too (store=True, readonly=False) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I considered this at the time of migration, but the incoterm
field is not owned by this module and adding a compute method for it does not scale to more than one module so I am not sure that is the best practice in this case. BTW there are plenty of leftover onchange methods in core Odoo.
No description provided.