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

[15.0][IMP] fieldservice_route: Add access rules and permissions for fsm_route, fsm_route_day and fsm_route_dayroute models #1297

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

Conversation

ppyczko
Copy link

@ppyczko ppyczko commented Jan 22, 2025

When managing fsm_route, fsm_route_day and fsm_route_dayroute records with the "User Own" group, I wasn't able to access any of the records nor the fsm_orders that were linked to a dayroute. This IMP adds access rules to ensure "User Own" members see only their assigned records, while managers have full access.

cc https://github.com/APSL 166041

@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @BernatObrador please review

@OCA-git-bot
Copy link
Contributor

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

…ute, fsm_route_day and fsm_route_dayroute models
@ppyczko ppyczko force-pushed the 15.0-imp-fieldservice_route-fsm-user-own branch from 3401264 to 0ce7ab9 Compare January 22, 2025 07:13
@ppyczko ppyczko marked this pull request as ready for review January 22, 2025 07:31
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 and code review

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