banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #01679
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