← Back to team overview

banking-addons-team team mailing list archive

Re: [Merge] lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-related-balance-check into lp:banking-addons/bank-statement-reconcile-7.0

 

Stéphane,

I understand your point, but that also has the weird point that the method that you need to write some code far away in class A to make a function field in class B.

My opinion is that there are a few ways to do that, but if we name variables and methods properly, and maybe add a docstring, IMO it's good enough.

Thanks!
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-related-balance-check/+merge/209723
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.


References