Code review comment for lp://qastaging/~akretion-team/multi-company/multi-company-action-user

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

I expect that the use of this module will become clear in future modules, so I only have a couple of nits:

- better to remove the empty lines at the end of __init__.py
- l.114..119 will probably look more elegant if you break the line after the parenthesis and align with the 'o' in 'automatic'.
- l. 127 you can use orm.except_orm instead, so that you do not have to import osv at all

review: Needs Fixing

« Back to merge proposal