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 account budget operating unit #53

Conversation

AaronHForgeFlow
Copy link
Contributor

=============================
Account Budget Operating Unit

This module introduces the following features:

  • Adds the OU as a required field in the budget plan / budget line

  • Security rules are defined to ensure that users can only display the
    budget in which they are allowed access to.

Depends on #50

operating_unit_default_get(self._uid),
)

@api.one
Copy link
Member

Choose a reason for hiding this comment

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

Hmm.. api.one is deprecated isn'it?
cc: @jbeficent

@AaronHForgeFlow AaronHForgeFlow force-pushed the 10.0-mig-account_budget_operating_unit branch from 9b35b67 to 1dfdc38 Compare January 27, 2017 12:51
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

👍

@AaronHForgeFlow AaronHForgeFlow force-pushed the 10.0-mig-account_budget_operating_unit branch from 66af9ae to 0fd366e Compare June 29, 2018 08:26
Copy link
Member

@serpentcs-dev1 serpentcs-dev1 left a comment

Choose a reason for hiding this comment

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

Code Review 👍

@AaronHForgeFlow AaronHForgeFlow force-pushed the 10.0-mig-account_budget_operating_unit branch from 0fd366e to d7b8a94 Compare March 7, 2019 16:40
@AaronHForgeFlow
Copy link
Contributor Author

Closing as nobody cares

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants