Skip to content

Conversation

@andreampiovesana
Copy link
Contributor

security for user_own and user_team groups otherwise user with different permissions are not able to set motive in a ticket

 security for user_own and user_team groups
@OCA-git-bot
Copy link
Contributor

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

@andreampiovesana andreampiovesana changed the title Update ir.model.access.csv for user_own and user_team groups [18.0] [FIX] helpdesk_motive add permission for user_own and user_team groups Jun 20, 2025
@dalonsod
Copy link
Contributor

Hello @andreampiovesana are really needed create and write permissions? And setting permissions for group_helpdesk_user_team is not required, as it actually inherits from group_helpdesk_user_own.

@andreampiovesana
Copy link
Contributor Author

andreampiovesana commented Jun 23, 2025

change as you want, but now a user_team can't set a motive

@dalonsod
Copy link
Contributor

Tested in runboat, and it works even if ACL for team group is removed, as expected. Please note that motives depend on team, you may tested sometimes creating a ticket selecting a team that could not have available motives. The only change that should be set is moving current ACL team for user group to user_own group (line 2 at .csv file).

And please remember the other question: why do you add create and write permissions? Only setting read permissions should be enough IMO

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 26, 2025
@andreampiovesana
Copy link
Contributor Author

merge?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants