Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

10.0 mig sales team operating unit #47

Merged

Conversation

LoisRForgeFlow
Copy link
Contributor

Sales Team Operating Unit

This module introduces the following features:

  • Adds the Operating Unit (OU) to the Sales Team.

  • The user's default Operating Unit (OU) is proposed at the time of creating
    the Sales Team.

  • Security rules are defined to ensure that users can only see the Sales Team
    of that Operating Units in which they are allowed access to.

@LoisRForgeFlow
Copy link
Contributor Author

depends on #45

@LoisRForgeFlow LoisRForgeFlow force-pushed the 10.0-mig-sales_team_operating_unit branch from c7e6ecb to 5225dc1 Compare January 27, 2017 09:36
# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
from odoo import api, fields, models, _
from odoo.exceptions import UserError

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lreficent unused, it is blocking flake8 can you please remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, sure

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It wasn't in that file. But yes, it should be ok now.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lreficent Sorry my mistake it has to removed from test cases but instead we removed from crm_team.py

So you need to add the import again to py file and remove it from test cases.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

solved.

@LoisRForgeFlow LoisRForgeFlow force-pushed the 10.0-mig-sales_team_operating_unit branch from 4019654 to bd59b8d Compare January 27, 2017 12:45
@LoisRForgeFlow
Copy link
Contributor Author

This seems fine. Can you review? @jbeficent @aheficent @mreficent

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@LoisRForgeFlow LoisRForgeFlow force-pushed the 10.0-mig-sales_team_operating_unit branch from 74e2b43 to f19385e Compare January 30, 2017 10:44
@JordiBForgeFlow JordiBForgeFlow merged commit 365dabb into OCA:10.0 Jan 30, 2017
@LoisRForgeFlow LoisRForgeFlow mentioned this pull request Jan 30, 2017
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants