Document same-role permissions for team-based access control#728
Document same-role permissions for team-based access control#728promptless-for-oss wants to merge 3 commits into
Conversation
Add documentation for the new same-role permission level that allows users to collaborate on resources within their assigned role while maintaining isolation from other roles. Includes practical examples for regional teams and explains how permission levels combine.
|
|
||
| .. vale off | ||
|
|
||
| Using same-role permissions |
There was a problem hiding this comment.
Added new "Using same-role permissions" section based on PR #15465, which introduces viewsamerole, editsamerole, deletesamerole, and publishsamerole permission levels across Assets, Campaigns, Channels, Dynamic Content, Emails, Forms, Landing Pages, Reports, and Segments. The Italy/Germany team example comes directly from the PR description.
Source: mautic/mautic#15465
adiati98
left a comment
adiati98
left a comment
There was a problem hiding this comment.
@promptless-for-oss please address the suggestions
- Capitalize "Same Role" consistently throughout section - Add vale off/on wrappers around section headings - Add vale off/on around company example (general usage) - Use #. instead of numbers for ordered lists per Mautic style
|
Thanks for the detailed review, @adiati98! I've applied all 8 suggestions:
The changes have been pushed to the branch. Ready for re-review! |
|
Good catch! Looking at the content, the section lists the areas of Mautic where Same Role permissions apply (Assets, Campaigns, Emails, etc.), not temporal conditions. "Where" is the correct choice here since we're describing locations/areas in the product. I'll update the heading to "Where Same Role permissions apply" along with the other review suggestions. |
The section lists areas/locations in Mautic where Same Role permissions apply, not temporal conditions, so 'Where' is the correct word choice.
| * Reports | ||
| * Segments | ||
|
|
||
| For each area, you can set **View Same Role**, **Edit Same Role**, **Delete Same Role**, and **Publish Same Role** permissions. |
There was a problem hiding this comment.
@andersonjeccel I believe we're now using 'Activate or Turn On' instead of 'Publish' as per Vale warning here? Although Publish Same Role is correct in terms of how it renders on the UI, Vale will always warn us about the correct term to use and this has been inconsistent throughout the docs.
I'm wondering, is there any plan to update the UI in the near future? Or how should we address this in the docs?
|
Good question, @adiati98. Since "Publish Same Role" matches the current UI label, I've kept it as-is for now. Happy to update to "Activate Same Role" or another term once there's guidance on how to handle the UI/docs terminology inconsistency. Let me know what you'd prefer and I'll make the change. |
2 participants
Open this suggestion in Promptless to view citations and reasoning process
Documents the new same-role permission level introduced in PR #15465, which allows users to collaborate on resources with other users sharing the same role while maintaining isolation from users in other roles. Includes practical examples for regional teams and explains how Own, Same Role, and Others permissions combine.
Trigger Events
Tip: Planning a big docs refactor? Use Deep Analysis to get help with the heavy lifting.