Review: Approve

Great work, thanks! There are a lot of nice improvements in this change. As for 
the payment mode type, removing its value in the XML declaration is the correct 
solution, indeed.

Looks like the recent integration with Launchpad translations caused a couple 
of conflicts, as Stéphane noticed already. Did you try and correct these in 
revision 209? If that is the case, it may be necessary to resolve these at 
merge time.


-- 
https://code.launchpad.net/~akretion-team/banking-addons/70-sepa-credit-transfer-update/+merge/194948
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.

-- 
Mailing list: https://launchpad.net/~banking-addons-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~banking-addons-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to