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

[14.0][IMP][FIX] sale_commission_product_criteria[_domain]: permission fixes #453

Merged
merged 2 commits into from
Sep 13, 2023

Conversation

aleuffre
Copy link

Small changes to fix some issues with permissions with these two modules.

For a quick test of the issue on the main branch:

  • Install the contacts application if not already installed
  • Login as Marc Demo
  • Try to access some contacts (for example: "Agent Rules Restricted Italy")

The error is You are not allowed to access "Commission.Item"

@OCA-git-bot
Copy link
Contributor

Hi @ilyasProgrammer,
some modules you are maintaining are being modified, check this out!

Copy link
Member

@ilyasProgrammer ilyasProgrammer left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

Functional ok!

@aleuffre aleuffre force-pushed the 14.0-commission-permission-fix branch from 4ffae5c to ffc6e6d Compare September 13, 2023 10:00
@ilyasProgrammer
Copy link
Member

ilyasProgrammer commented Sep 13, 2023

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-453-by-ilyasProgrammer-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4f0273e into OCA:14.0 Sep 13, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 32b06e7. Thanks a lot for contributing to OCA. ❤️

@aleuffre aleuffre deleted the 14.0-commission-permission-fix branch October 31, 2023 08:36
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.

4 participants