-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[19.0][MIG] sale_advance_payment: Migration to 19.0 #3942
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
base: 19.0
Are you sure you want to change the base?
[19.0][MIG] sale_advance_payment: Migration to 19.0 #3942
Conversation
… better support in multi-currency.
Currently translated at 92.1% (35 of 38 strings) Translation: sale-workflow-14.0/sale-workflow-14.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_advance_payment/fr/
- Allow to make advance payments when invoice has been created. This is justified by the fact that users may want to create a payment reversal in some circumstances. - Introduce payment type (inbound, outbound) to improve usability
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-14.0/sale-workflow-14.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_advance_payment/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-15.0/sale-workflow-15.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_advance_payment/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-15.0/sale-workflow-15.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_advance_payment/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-16.0/sale-workflow-16.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_advance_payment/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-17.0/sale-workflow-17.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_advance_payment/
Currently translated at 100.0% (43 of 43 strings) Translation: sale-workflow-17.0/sale-workflow-17.0-sale_advance_payment Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_advance_payment/fr/
Otherwise, we might get incorrect residuals when we get refunds
- Filter out cancelled invoices/refunds - Use invoice currency amounts instead of company currency amounts
c05ad24 to
930b213
Compare
|
/ocabot migration sale_advance_payment |
930b213 to
07cc2e1
Compare
| if not sale_ids: | ||
| return res | ||
| sale_id = fields.first(sale_ids) | ||
| sale_id = sale_ids[0] |
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.
Why this change ?
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.
fields.first() has been removed in Odoo 19.0, so it’s no longer available.
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.
Ooooh, so bad.
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.
Suggestion is to use next(iter( instead.
odoo/odoo@871427d#diff-845fca2f1c693995c343fa58c47c296e5aa5e7769aea6f83859a8854f47959eeR20
No description provided.