Skip to content

Allow per-model permission level management#2656

Open
zipavlin wants to merge 2 commits into
area17:3.xfrom
zipavlin:3.x
Open

Allow per-model permission level management#2656
zipavlin wants to merge 2 commits into
area17:3.xfrom
zipavlin:3.x

Conversation

@zipavlin

Copy link
Copy Markdown
Contributor

Description

This PR modifies advance permission managements so it is possible to set permission level per module, not only on global level. This is useful when there is no need for per-item management except for few specific modules, fe. pages where we want to allow users to edit only specific page(s).

This change should be backwards compatible and can be used by adding a level to twill.permissions.modules array:

'permissions' => [
    'level' => \A17\Twill\Enums\PermissionLevel::LEVEL_ROLE_GROUP,
    'modules' => [
        'pages' => \A17\Twill\Enums\PermissionLevel::LEVEL_ROLE_GROUP_ITEM,
        'posts'
    ]
]

Tests

I was not able to run tests on locale machine, but I did test it manually :)

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.

2 participants