-
-
Notifications
You must be signed in to change notification settings - Fork 693
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_invoicing_mode: Migration to 17.0 #1699
[17.0][MIG] partner_invoicing_mode: Migration to 17.0 #1699
Conversation
730be44
to
61b31ae
Compare
61b31ae
to
3c21fb0
Compare
3c21fb0
to
72b9be5
Compare
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.
@NICO-SOLUTIONS I think you should double-check against this migration. You can check at #1515 and #1295.
@NICO-SOLUTIONS You missed a lot of commit history. Please rebase the commit and migration against it. |
[UPD] Update partner_invoicing_mode.pot [UPD] README.rst partner_invoicing_mode 16.0.1.0.1
Translated using Weblate (Spanish) Currently translated at 100.0% (8 of 8 strings) Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/es/ [UPD] README.rst
Translated using Weblate (Italian) Currently translated at 100.0% (8 of 8 strings) Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/it/
Allow to have a better extension mechanism when having multiple invoicing modes.
The field one_invoice_per_order wasn't used correctly. So, now, create invoices using the partner option by calling the grouping option on sale orders
To improve flexibility, the invoicing grouping option could be different on the sale order level than on the partner level.
As the read_group() can aggregate sale order ids per partner, use the ids list instead.
…al field As a company can have several different invoicing address, invoicing can occur at different dates.
…tion As validation jobs could last long time if there are plenty, ensure that the wanted date is correctly set.
In order to be able to benefits from standard grouping function, override it and add missing keys. [UPD] Update partner_invoicing_mode.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/ Translated using Weblate (Italian) Currently translated at 94.4% (17 of 18 strings) Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/it/ Translated using Weblate (Italian) Currently translated at 100.0% (18 of 18 strings) Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/it/ Translated using Weblate (Spanish) Currently translated at 100.0% (18 of 18 strings) Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode/es/
72b9be5
to
d0b1f4d
Compare
d0b1f4d
to
a6c899c
Compare
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.
LGTM 👍
/ocabot migration partner_invoicing_mode |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 5be7298. Thanks a lot for contributing to OCA. ❤️ |
standard migration