openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #02394
Re: lp:~jean-lelievre/stock-logistic-report/logistics_list_report into lp:stock-logistic-report
Review: Needs Fixing code review, no tests
Thanks for proposing your module
To ensure it is accepted in community addons, please ensure your code respect community coding conventions:
- PEP8
- Make sure context are propagated.
- Add a pot file
- osv.osv_memory -> orm.TransientModel
https://doc.openerp.com/contribute/05_developing_modules/#community-guidelines
Cheers
--
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/logistics_list_report/+merge/182038
Your team Stock and Logistic Core Editors is subscribed to branch lp:stock-logistic-report.