Code review comment for lp:~therp-nl/account-banking/upgrade-6.0

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Pieter,

Your backports have been merged in this branch.

I have kept assigning code 'UNKNOW' to unknown banks. The reference to res_bank is required for res_partner_banks in 6.0. Without this modification a multitude of unknown banks can get accumulated.

You might still want to look at the following changes for backporting to v5:

account_banking/wizard/bank_import.py @@ -801,7 +802,7 @@
account_banking/sepa/online.py @@ -134,7 +134,7 @@

Regards,
Stefan.

review: Needs Resubmitting

« Back to merge proposal