Merge lp://qastaging/~akretion-team/multi-company/multi-company-action-user into lp://qastaging/multi-company
Proposed by
Benoit Guillot - http://www.akretion.com
Status: | Needs review |
---|---|
Proposed branch: | lp://qastaging/~akretion-team/multi-company/multi-company-action-user |
Merge into: | lp://qastaging/multi-company |
Diff against target: |
156 lines (+136/-0) 4 files modified
multi_company_action_user/__init__.py (+23/-0) multi_company_action_user/__openerp__.py (+41/-0) multi_company_action_user/res_company.py (+43/-0) multi_company_action_user/res_company_view.xml (+29/-0) |
To merge this branch: | bzr merge lp://qastaging/~akretion-team/multi-company/multi-company-action-user |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alexandre Fayolle - camptocamp | Needs Resubmitting | ||
Ana Juaristi Olalde (community) | Approve | ||
Pedro Manuel Baeza | Needs Information | ||
Joël Grand-Guillaume @ camptocamp | code review, no tests | Approve | |
Stefan Rijnhart (Opener) | Approve | ||
Review via email: mp+179179@code.qastaging.launchpad.net |
Description of the change
Hello,
I propose a new module in this branch : multi_company_
This module is a generic module. It adds a field on the company : automatic_
This field defines a user for a company that will be used in the code for intercompany automatic actions. For instance, automatic intercompany invoices, sales ...
Best regards.
Benoît
To post a comment you must log in.
Unmerged revisions
- 6. By Benoit Guillot - http://www.akretion.com
-
[FIX] clean code
- 5. By Benoit Guillot - http://www.akretion.com
-
[ADD] new module to add a specific user to the company to make actions on this company
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