Code review comment for lp:~therp-nl/banking-addons/ba70-aggregate_payment

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

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.

review: Needs Information

« Back to merge proposal