banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00874
Re: lp:~savoirfairelinux-openerp/banking-addons/loose-coupling into lp:banking-addons/bank-statement-reconcile-7.0
Some quick observations, which might be useful.
1. 'auto_install' in __openerp__.py
2. def _default_customer_payment_request_info can be written in one line.
3. 'state' field should be kept readonly!
4. some methods you write need to pass context within(create, write, etc.)
Thanks.
--
https://code.launchpad.net/~savoirfairelinux-openerp/banking-addons/loose-coupling/+merge/185033
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~savoirfairelinux-openerp/banking-addons/loose-coupling into lp:banking-addons/bank-statement-reconcile-7.0.
References