banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #01965
Re: [Merge] lp:~acsone-openerp/banking-addons/bank-statement-reconcile-70-multi-statements into lp:banking-addons/bank-statement-reconcile-7.0
Hi Yannick,
The comment has been fixed and a docstring added to multi_statement_import.
> l.44 you can use self.pool[''] instead of self.pool.get('')
self.pool.get is used in a lot of places... Do you want it to be replaced everywhere? It's just a copy/paste of l.82
> l.47 + l.53 > 80 characters
Do we need to pay attention to this pep8 rule?
Regards,
lmi
--
https://code.launchpad.net/~acsone-openerp/banking-addons/bank-statement-reconcile-70-multi-statements/+merge/219683
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.
References