-
-
Notifications
You must be signed in to change notification settings - Fork 366
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] partner_delivery_zone #769
Conversation
…rs and pickings by delivery zones
[IMP] partner_delivery_zone: Get delivery zone from commercial partner for non delivery contacts
Currently translated at 100.0% (18 of 18 strings) Translation: delivery-carrier-15.0/delivery-carrier-15.0-partner_delivery_zone Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-15-0/delivery-carrier-15-0-partner_delivery_zone/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-partner_delivery_zone Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-partner_delivery_zone/
Currently translated at 100.0% (19 of 19 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-partner_delivery_zone Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-partner_delivery_zone/es/
- Include context keys for avoiding mail operations overhead.
Please put the proper PR title, as I did. If not, your PR may not be found by others, as the module name is not correctly put. Check also the CI. |
Ok thanks! |
/ocabot migration partner_delivery_zone |
Please squash the last 3 migration commits together |
[MIG] partner_delivery_zone: Migration to 17.0 [MIG] partner_delivery_zone: Migration to 17.0
40f6f7d
to
d5b66c5
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
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.
Please include #843
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.
Please include #858
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
Please include #911 |
@pedroguirao Thanks for this. Could you follow migration guide ? https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0#how-to In fact, migration changes (version number, ...) should appear in migration commit. Thanks |
@pedroguirao can you look at the comments? |
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.
Please, change the sale.order filter to readonly property
<field name="inherit_id" ref="sale.view_order_form" /> | ||
<field name="arch" type="xml"> | ||
<xpath expr="//field[@name='payment_term_id']" position="after"> | ||
<field name="delivery_zone_id" readonly="state in ['done', 'cancel']" /> |
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.
<field name="delivery_zone_id" readonly="state in ['done', 'cancel']" /> | |
<field name="delivery_zone_id" readonly="state in ['sale', 'cancel']" /> |
<xpath expr="//field[@name='company_id']" position="before"> | ||
<field | ||
name="delivery_zone_id" | ||
readonly="state in ['done', 'cancel']" |
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.
readonly="state in ['done', 'cancel']" | |
readonly="state in ['sale', 'cancel']" |
supersede #995 |
No description provided.