banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00814
Re: lp:~camptocamp/banking-addons/bank-statement-reconcile-70-transaction_ref into lp:banking-addons/bank-statement-reconcile-70
Review: Needs Information code review
Hi,
It would be great to elaborate a bit the functional description / use case in the module descriptor, as the purpose of the module is not immediately obvious.
Is there any particular reason there are two different classes in the module both inheriting from 'easy.reconcile.advanced'?
Otherwise I noticed a few pep8 warnings in account.py.
-sbi
--
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-70-transaction_ref/+merge/179891
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~camptocamp/banking-addons/bank-statement-reconcile-70-transaction_ref into lp:banking-addons/bank-statement-reconcile-70.
Follow ups
References