← Back to team overview

banking-addons-team team mailing list archive

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