openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #02715
Re: [Merge] lp:~camptocamp/project-reporting/7.0-fix-dissociate-lep into lp:project-reporting
Thanks for your review Alexandre.
1. We changed the key passed in "vals", but the key in the context is still the same. So that LGTM.
2. I followed http://www.python.org/dev/peps/pep-0257/#multi-line-docstrings "Insert a blank line before and after all docstrings (one-line or multi-line) that document a class"
--
https://code.launchpad.net/~camptocamp/project-reporting/7.0-fix-dissociate-lep/+merge/200695
Your team Project Core Editors is subscribed to branch lp:project-reporting.
References