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

[9.0] extract analytic_base_department #170

Closed
wants to merge 31 commits into from

Conversation

yvaucher
Copy link
Member

Moving OCA/department#56 here

jgrandguillaume and others added 26 commits April 27, 2016 19:26
(lp:c2c-addons/6.1  rev 28.3.24)
(lp:c2c-addons/6.1  rev 40.1.1)
(lp:c2c-addons/6.1  rev 74)
…w group and search on both user's or account's related department
[FIX] Context propagation
[IMP] Few PEP8
In order to get visibility on https://www.odoo.com/apps the OCA board has
decided to add the OCA as author of all the addons maintained as part of the
association.
@yvaucher yvaucher force-pushed the 9.0-extract-analytic_base_department branch 4 times, most recently from 0905319 to bc2ece6 Compare April 28, 2016 07:32
@yvaucher yvaucher force-pushed the 9.0-extract-analytic_base_department branch 2 times, most recently from 6c2c8b8 to 4740e97 Compare April 28, 2016 08:16
@yvaucher yvaucher force-pushed the 9.0-extract-analytic_base_department branch from 4740e97 to d3882df Compare April 28, 2016 08:47
@coveralls
Copy link

coveralls commented Apr 28, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d3882df on yvaucher:9.0-extract-analytic_base_department into 00eb002 on OCA:9.0.

@coveralls
Copy link

coveralls commented Apr 28, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling a811713 on yvaucher:9.0-extract-analytic_base_department into 00eb002 on OCA:9.0.

@dreispt
Copy link
Sponsor Member

dreispt commented May 1, 2016

@yvaucher Maybe this PR is more appropriate for OCA/contract.

@yvaucher
Copy link
Member Author

yvaucher commented May 2, 2016

@dreispt Maybe more in https://github.com/oca/account-analytic ? As we already have a dependency on this repo in oca_requirements.txt ?

@dreispt
Copy link
Sponsor Member

dreispt commented May 2, 2016

Sounds good to me.
Maybe we should also check with some of the contributors there.
@sbidoul @lepistone @guewen : does adding departments to analytic account fit well into the OCA/account-analytic repo?

@yvaucher
Copy link
Member Author

Moved in OCA/account-analytic#57

@yvaucher yvaucher closed this May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants