-
-
Notifications
You must be signed in to change notification settings - Fork 584
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
[12.0] FIX pos_partner_firstname allowing non admin users to get partner_names_order from POS frontend #445
[12.0] FIX pos_partner_firstname allowing non admin users to get partner_names_order from POS frontend #445
Conversation
…es_order from POS frontend Otherwise they would get access error on ir.config_parameter Steps: - Create a user POS manager without administration rights - Using that user, open POS frontend
Hi @robyf70, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional test ok.
Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the patch.
/ocabot merge minor
I think you need to run |
Sorry @eLBati you are not allowed to merge. To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons. If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the |
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at d6f7ee8. Thanks a lot for contributing to OCA. ❤️ |
Otherwise they would get access error on ir.config_parameter
Steps: