banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00474
Re: [Merge] lp:~camptocamp/banking-addons/improve_lookup into lp:banking-addons/bank-statement-reconcile-70
Hello,
Thank for your comments
I think I will add constraints on the bank_label_field this will be more explicit to the end user. What do you think.
line 23: The memoizer is passed trough each line of the statement it is not locally scoped as the function is called for each line of the statement. That's why the previous implementation was innefficient.
line 101. Tho.. finger crossed, I wanted to remove the get usage. That why we made code review ;)
--
https://code.launchpad.net/~camptocamp/banking-addons/improve_lookup/+merge/154730
Your team Banking Addons Team is subscribed to branch lp:banking-addons/bank-statement-reconcile-70.
References