Code review comment for lp:~savoirfairelinux-openerp/banking-addons/loose-coupling

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

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.

« Back to merge proposal