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

[FIX] maintenance: Add missing ACL for maintenance.team #32966

Closed
wants to merge 1 commit into from

Conversation

zaoral
Copy link
Contributor

@zaoral zaoral commented Apr 25, 2019

Description of the issue/feature this PR addresses:

This error has been already fixed in branch 12.0, this commit only try to backport the fix to branch 11.0.

Current behavior before PR:

Before this PR if only the Administrator user is able to create/edit/delete maintenance teams.

Desired behavior after PR is merged:

After this PR the Equipment Manager will be able to create/edit/delete the maintenance teams as it does in version 12.0

NOTE:

Also add another missing ACL for mail_activity that I found when comparing the ACL csv file between version 11.0 and 12.0 branchs. Here are the two commits that introduce this fixes in version 12:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Before this PR if only the Administrator user is able to create/edit/delete
maintenance teams.

After this PR the Equipment Manager will be able to create/edit/delete the
maintenance teams.

This fix has been already fixed in branch 12.0, this commit only try to
backport this fix to branch 11.0.

Also add another missing ACL fro mail_activity that I found when I make
the a diff of the csv file between version 11.0 and 12.0

Here are the two commits that introduce this fixes in version 12:

* odoo@d0e2cae
* odoo@a0e7c0d
@zaoral zaoral changed the title [ADD] maintenance: Add missing ACL for maintenance.team [FIX] maintenance: Add missing ACL for maintenance.team Apr 25, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 25, 2019
@mart-e mart-e added the Logistics about stock, mrp, delivery, purchase label Jul 26, 2019
@mart-e mart-e requested a review from sle-odoo July 26, 2019 10:11
@mart-e
Copy link
Contributor

mart-e commented Jul 26, 2019

But the commits you are referring to change other things than just adding ACL.
Could you detail what bug you are fixing exactly please?

@zaoral zaoral closed this Jan 22, 2021
@zaoral zaoral deleted the 11.0-h-17744-kz branch January 22, 2021 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses Logistics about stock, mrp, delivery, purchase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants