[18.0][MIG] account_payment_partner: Migration to version 18.0#1439
Merged
OCA-git-bot merged 116 commits intoOCA:18.0from Apr 16, 2025
Merged
[18.0][MIG] account_payment_partner: Migration to version 18.0#1439OCA-git-bot merged 116 commits intoOCA:18.0from
OCA-git-bot merged 116 commits intoOCA:18.0from
Conversation
carlos-lopez-tecnativa
force-pushed
the
18.0-mig-account_payment_partner
branch
from
5be06f2 to
befd64b
Compare
3 tasks
Member
|
/ocabot migration account_payment_partner |
sbidoul
reviewed
Apr 2, 2025
|
Shouldn't the dependency be added to the module that creates "mandate_id" ?
The module that creates the field in v17 is "account_banking_mandate" |
Member
|
@sergioM-S73 yes, that part should be extracted outside this one. We will do it after merging |
- account_payment_partner - account_payment_sale - account_payment_sale_stock - account_payment_purchase Filter the selection of invoices per payment type. Add active field on payment.mode and payment.mode.type. Add menu entry for Payment Types.
* Migration and enhancement of all modules involved in SEPA * Clean files * Replace deprecated workflow calls * Translation template files * Translations to spanish * Use sale_ok and purchase_ok instead of tweaking payment type
* IMP: Add filter group by payment mode in account invoices * FIX: Remove deprecated tag type
* move unused file + cleanup code * Add sale_ok and purchase_ok filters in partner/sale/purchase form views * Use widget=selection for payment_mode_id fields * Update demo data * Return res in inherit (even if res is empty in this case)
…hout invoice (manual entries in receivables/payables)
FIX tabs FIX translate
* Forgot to remove a usecase of debit_denied * Add ACL * Better filters on payment.order.create wizard * Add default values for those filters on payment.mode * FIX remove coma at end of line * Add translatable 'label' field on payment.mode, designed to be used in invoice report
* Short headers * Remove label field There's already a field in the payment mode called 'note' that is printed on the invoices, so there's no need of another field for that purpose. This field is added by account_banking_payment_export * Bump version numbers * es translations * Name to filter for better inheratiblity
… a lot of improvements) during the Sorrento Code sprint 2016 Improvements include: - full re-organisation of modules and big re-organisation of the code - simplification of the code related to the fact that support for direct debit is now in t he base module, not added by an optional module account_direct_debit (module was removed) - new design of the wizard to select move lines to pay - support for non-SEPA file transfer- - support for German direct debit SEPA files (fixes bug OCA#129) - remove workflow of payment.order * Finalise the wizard of selection of move lines to pay * Add button "Add to payment/debit order" on invoice form view * Started to integrate payment transfer in account_payment_order (not finished at all though) * Copy payment mode and mandate from invoice to refund * Run dos2unix on the few files that had DOS carriage return * Add _id suffix on M2O fields customer_payment_mode and supplier_payment_mode (Odoo did the same on all the M2O property fields of res.partner) * Better handling of partner_bank_id and mandate_id on invoice: invisible/required/onchange * Move field bank_account_required from module account_payment_partner to account_payment_mode
Currently translated at 100.0% (37 of 37 strings) Translation: bank-payment-16.0/bank-payment-16.0-account_payment_partner Translate-URL: https://translation.odoo-community.org/projects/bank-payment-16-0/bank-payment-16-0-account_payment_partner/fr/
Currently translated at 100.0% (37 of 37 strings) Translation: bank-payment-17.0/bank-payment-17.0-account_payment_partner Translate-URL: https://translation.odoo-community.org/projects/bank-payment-17-0/bank-payment-17-0-account_payment_partner/it/
…ions in HTML module descriptions are deprecated since Odoo 17
Currently translated at 100.0% (37 of 37 strings) Translation: bank-payment-17.0/bank-payment-17.0-account_payment_partner Translate-URL: https://translation.odoo-community.org/projects/bank-payment-17-0/bank-payment-17-0-account_payment_partner/sv/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (37 of 37 strings) Translation: bank-payment-17.0/bank-payment-17.0-account_payment_partner Translate-URL: https://translation.odoo-community.org/projects/bank-payment-17-0/bank-payment-17-0-account_payment_partner/it/
…invoices This error is happening because the salesperson has not access to payment_method_id. With this changes we avoid the error and the invoices are printed like normally. TT31296
…partner_multi_company)
…fault payment method in contacts.
carlos-lopez-tecnativa
force-pushed
the
18.0-mig-account_payment_partner
branch
from
befd64b to
e84f27e
Compare
…_mandate TODO: move this code to the account_banking_mandate module
pedrobaeza
approved these changes
Apr 16, 2025
Member
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
As for now, there seems to be a "fork", we don't dig more on the fields UI, just the current visibility group over core fields. If this changes in the future, we can open improvement PRs.
/ocabot merge nobump
Contributor
|
On my way to merge this fine PR! |
Contributor
|
Congratulations, your PR was merged at 3b6324b. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
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.
Supersedes #1415
Depends on:
TT51707
@Tecnativa @pedrobaeza @victoralmau @etobella @JordiMForgeFlow @carlosdauden could you please review this?