Review: Needs Fixing

2624: that should be openerp.tools or openerp.tools.misc, right?

Minor thing:

2169ff: I think you can avoid the awkwardness by using 
banking_import_transaction._confirm_payment_order (that is, the reference to 
the unbound method)

-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba7.0-RFR-split_off_payment_part/+merge/153680
Your team Banking Addons Team is subscribed to branch 
lp:~banking-addons-team/banking-addons/ba70-mig_account_iban_preserve_domestic.

-- 
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