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

[MIG] Migrate hr_security to V8. #96

Merged
merged 6 commits into from
Jun 2, 2015
Merged

Conversation

feketemihai
Copy link
Member

No description provided.

@max3903 max3903 added this to the 8.0 milestone Apr 22, 2015
@max3903 max3903 mentioned this pull request Apr 22, 2015
58 tasks
@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 8feda95 on feketemihai:mig_hr_security into a96858e on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 8feda95 on feketemihai:mig_hr_security into a96858e on OCA:8.0.

@feketemihai
Copy link
Member Author

@max3903 Can you check this, the module adds some rules to hr.contract for employee and officer, the ideea is that from hr.contract the contracts button on employee and contracts menu are restricted to HR Manager group, should i modify this restrictions to allow viewing contract based on the rules specified?

@ghost
Copy link

ghost commented Apr 22, 2015

@feketemihai Yes, it would make sens.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 2b3c87f on feketemihai:mig_hr_security into a96858e on OCA:8.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 2b3c87f on feketemihai:mig_hr_security into a96858e on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 2b3c87f on feketemihai:mig_hr_security into a96858e on OCA:8.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.59% when pulling 2b3c87f on feketemihai:mig_hr_security into a96858e on OCA:8.0.

@mdietrichc2c
Copy link

LGTM, 👍

@max3903
Copy link
Sponsor Member

max3903 commented Jun 2, 2015

👍

max3903 pushed a commit that referenced this pull request Jun 2, 2015
[MIG] Migrate hr_security to V8.
@max3903 max3903 merged commit 42439f0 into OCA:8.0 Jun 2, 2015
@feketemihai feketemihai deleted the mig_hr_security branch November 11, 2015 05:15
sambarros pushed a commit to sambarros/hr that referenced this pull request Jul 26, 2018
Mraimou pushed a commit to camptocamp/hr that referenced this pull request Nov 25, 2019
BSIBSO-1008: fix to retrieve supplier price
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