Review: Needs Information

Is there a reason why you didn't use the account_move_line.reconcile() / 
account_move_line.partial_reconcile()?
I think there is one, but I'm curious to know why you couldn't use it 
(performance?).

Aside of this question, from a code point of view, sounds good to me.
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-aggregate_payment/+merge/178466
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