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 analytic operating unit #52

Merged
merged 2 commits into from Jan 31, 2017

Conversation

LoisRForgeFlow
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow commented Jan 26, 2017

Analytic Operating Unit

This module allows users to introduce into the analytic accounts the allowed
operating units. Only a user will permissions to operate on the operating
units indicated in an analytic account will have the rights to use it.

Depends on #45

@LoisRForgeFlow
Copy link
Contributor Author

Ready to merge I think. Can you review?

cc @aheficent @jbeficent @darshan-serpent @mreficent @guewen


.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
:alt: Try me on Runbot
:target: https://runbot.odoo-community.org/runbot/213/9.0
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Change to 10

@@ -0,0 +1,16 @@
# -*- coding: utf-8 -*-
# © 2015-17 Eficent Business and IT Consulting Services S.L.
# © 2015 Serpent Consulting Services Pvt. Ltd.
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Change to:
Copyright 2016-17

For all contributors

# © 2016-17 Eficent Business and IT Consulting Services S.L.
# © 2016-17 Serpent Consulting Services Pvt. Ltd.
# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
from odoo import fields, models
Copy link
Contributor

Choose a reason for hiding this comment

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

From my point of view the travis fail is depending on this. Before, it was importing from openerp and all checks were green and runbot was fine. The same with #47, #48 and #46

Copy link
Contributor

Choose a reason for hiding this comment

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

Well I'm not sure about that.

The thing is that travis odoo test is showing for those Pull request the same error:

image

Any idea about how to fix this?

It's also appearing here: #51

Copy link
Member

Choose a reason for hiding this comment

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

In sales_team_operating_unit PR : #47 the travis was green before the commit which removed the data tags from the view files. And now it is showing the same error in travis like the screenshot shared above by Aaron.

I am not sure why this is happening, do you guys have any idea regarding this?!

cc @lreficent @aheficent @mreficent

@LoisRForgeFlow LoisRForgeFlow force-pushed the 10.0-mig-analytic_operating_unit branch 3 times, most recently from 8ad0aff to 52700a4 Compare January 30, 2017 10:13
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.

👍

@JordiBForgeFlow JordiBForgeFlow merged commit 3160f2c into OCA:10.0 Jan 31, 2017
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

4 participants