-
-
Notifications
You must be signed in to change notification settings - Fork 531
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
[16.0][MIG] account_banking_mandate_sale #1100
Merged
OCA-git-bot
merged 39 commits into
OCA:16.0
from
akretion:16-mig-account_banking_mandate_sale-improve
Aug 15, 2023
Merged
[16.0][MIG] account_banking_mandate_sale #1100
OCA-git-bot
merged 39 commits into
OCA:16.0
from
akretion:16-mig-account_banking_mandate_sale-improve
Aug 15, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add option 'mandate_required' on payment orders Autoselect first valid mandate on customer invoice that have a payment mode 'mandate_required' = True Add option on select move lines to pay wizard to allow selection of litigation moves (unchecked by default), in order to integrate the feature of the module account_payment_blocking
…yment method Display chatter on payment orders Several small usability improvements
Port almost all modules to v10 * Update to EPC Rulebook v9.2 that start to apply on 2016-11-20 (bug OCA#300)
…or the partner Closes OCA#459
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/
Currently translated at 100.0% (4 of 4 strings) Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/pt_BR/
Currently translated at 33.3% (2 of 6 strings) Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/sl/
Currently translated at 100.0% (6 of 6 strings) Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/sl/
Currently translated at 100.0% (6 of 6 strings) Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-payment-12.0/bank-payment-12.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-12-0/bank-payment-12-0-account_banking_mandate_sale/
Currently translated at 87.5% (7 of 8 strings) Translation: bank-payment-14.0/bank-payment-14.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-14-0/bank-payment-14-0-account_banking_mandate_sale/ca/
The mandate must now be linked to the invoicing partner and not to the partner of the sale order
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-payment-15.0/bank-payment-15.0-account_banking_mandate_sale Translate-URL: https://translation.odoo-community.org/projects/bank-payment-15-0/bank-payment-15-0-account_banking_mandate_sale/
Remove inherit of sale.advance.payment.inv: no need to inherit _prepare_invoice_values() because it calls _prepare_invoice() from the sale.order object
Remove inherit of sale.advance.payment.inv: no need to inherit _prepare_invoice_values() because it calls _prepare_invoice() from the sale.order object
astirpe
approved these changes
Jun 20, 2023
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at ec2d36c. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PR improves the initial migration PR #1008
It also removes unneeded code in account_payment_sale: no need to inherit sale.advance.payment.inv because its method _prepare_invoice_values() calls _prepare_invoice(), cf
https://github.com/odoo/odoo/blob/16.0/addons/sale/wizard/sale_make_invoice_advance.py#L263
(which is great by the way !)