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

[17.0][IMP] pos_user_restriction: Restrict POS report orders #1322

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

peluko00
Copy link

@peluko00 peluko00 commented Feb 28, 2025

@OCA-git-bot
Copy link
Contributor

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

@peluko00 peluko00 changed the title [IMP] pos_user_restriction: Restrict POS report orders [17.0][IMP] pos_user_restriction: Restrict POS report orders Feb 28, 2025
Copy link

@BernatObrador BernatObrador left a comment

Choose a reason for hiding this comment

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

LGTM! Code review and tested in runboat

Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM, code review

@peluko00 peluko00 force-pushed the 17.0-imp-pos_user_restriction branch 2 times, most recently from 54d19db to eff029e Compare March 4, 2025 05:49
@peluko00
Copy link
Author

peluko00 commented Mar 4, 2025

Hi @eLBati, sorry for ping you here. Can you review please

@peluko00 peluko00 force-pushed the 17.0-imp-pos_user_restriction branch from eff029e to e8d6d05 Compare March 5, 2025 13:43
@peluko00 peluko00 force-pushed the 17.0-imp-pos_user_restriction branch from e8d6d05 to cb9101e Compare March 5, 2025 13:45
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.

5 participants