← Back to team overview

banking-addons-team team mailing list archive

Re: [Merge] lp:~therp-nl/banking-addons/ba70-aggregate_payment into lp:banking-addons

 

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.


References