bank-payment icon indicating copy to clipboard operation
bank-payment copied to clipboard

Published 20 hours ago • verified publisher icon OCA

Migration to version 15.0

Open OCA-git-bot opened this issue 3 years ago • 10 comments

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-15.0

Modules to migrate

  • [x] account_banking_mandate - By @manuelregidor - #914
  • [ ] account_banking_mandate_sale - https://github.com/OCA/bank-payment/pull/884
  • [x] account_banking_pain_base - By @manuelregidor - #912
  • [x] account_banking_sepa_credit_transfer - By @manuelregidor - #916
  • [x] account_banking_sepa_direct_debit - By @manuelregidor - #915
  • [ ] account_invoice_select_for_payment
  • [x] account_payment_mode - By @misern2 - #870
  • [x] account_payment_order - By @misern2 - #876
  • [ ] account_payment_order_return
  • [x] account_payment_partner - By @misern2 - #873
  • [x] account_payment_purchase - By @aisopuro - #952
  • [ ] account_payment_purchase_stock - https://github.com/OCA/bank-payment/pull/878
  • [x] account_payment_sale - By @manuelregidor - #913

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

OCA-git-bot avatar Oct 17 '21 10:10 OCA-git-bot

account_payment_mode #870

misern2 avatar Dec 11 '21 08:12 misern2

account_payment_partner #873

misern2 avatar Dec 11 '21 08:12 misern2

account_payment_order #876

misern2 avatar Dec 15 '21 09:12 misern2

account_payment_purchase https://github.com/OCA/bank-payment/pull/877

koenloodts avatar Dec 16 '21 14:12 koenloodts

account_payment_purchase_stock https://github.com/OCA/bank-payment/pull/878

koenloodts avatar Dec 16 '21 16:12 koenloodts

account_payment_sale https://github.com/OCA/bank-payment/pull/880

koenloodts avatar Dec 17 '21 14:12 koenloodts

I'm currently working on porting account_banking_pain_base to v15. I wanted to update the CodeList because I received information from a bank that they needed the category purpose RPRE which is not in the current list. So I was in the process of updating the code list for "purpose" and "category_purpose". But, in the latest version of the codeset (cf https://www.iso20022.org/catalogue-messages/additional-content-messages/external-code-sets), the purpose field (code set = "ExternalPurpose1Code") has 323 entries ! The list provided in the fields.Selection for the field "purpose" was already quite big (95 entries)... but now it's 323 entries. I think a fields.Selection is not appropriate for 323 entries ; you can't do a name_search in it, etc. So I'm planning to convert the fields "purpose" and "category_purpose" to a m2o field (and write some mig scripts). If you disagree with this change, please SAY SO NOW (this change is not fun and takes quite some time, so if many of you are against it I prefer not to do the work at all).

alexis-via avatar Jan 05 '22 11:01 alexis-via

I see it correct. Another possible improvement in the payment order is to have a store=False field to set them, and propagate to the existing lines. Remember to put the active mechanism for disabling them.

pedrobaeza avatar Jan 05 '22 15:01 pedrobaeza

It was not fun, but I did it: https://github.com/OCA/bank-payment/pull/885 I'll migrate account_banking_sepa_credit_transfer and account_banking_sepa_direct_debit tomorrow or in the coming days (some changes will be needed in those modules too to use the new code lists).

alexis-via avatar Jan 05 '22 23:01 alexis-via

account_payment_purchase https://github.com/OCA/bank-payment/pull/928 based on https://github.com/OCA/bank-payment/pull/877

airlessproject avatar May 20 '22 11:05 airlessproject