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.

Sign up for GitHub

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

Jump to bottom

Support new Drupal manage permissions tab #218

Closed
andybroomfield opened this issue Jun 24, 2022 · 1 comment
Closed

Support new Drupal manage permissions tab #218

andybroomfield opened this issue Jun 24, 2022 · 1 comment

Comments

@andybroomfield
Copy link
Contributor

See article and Drupal documentation.

We should support the manage permissions form. It requires making sure permissions are assigned to an entity. This can be done using the new BundlePermissionHandlerTrait but this only exists in Drupal 9.3. We could either copy the trait into the permission callback to keep compatibility, or we could start a 2.x branch now and start supporting new Drupal features.
andybroomfield added a commit that referenced this issue Jun 24, 2022
@andybroomfield
Initial support for new permissions form
5264a00 
Ref #218

Add new annotation to get the alert banner permissions form on the manage alert banner type
entity.
Also changes the alert banner permissions generator to make sure are assigned to the alert
banner entity using BundlePermissionHandlerTrait
@andybroomfield andybroomfield mentioned this issue Jun 24, 2022
Merged
andybroomfield added a commit that referenced this issue May 14, 2023
@andybroomfield
Initial support for new permissions form
4fd64cc 
Ref #218

Add new annotation to get the alert banner permissions form on the manage alert banner type
entity.
Also changes the alert banner permissions generator to make sure are assigned to the alert
banner entity using BundlePermissionHandlerTrait
andybroomfield added a commit that referenced this issue May 15, 2023
@andybroomfield
Initial support for new permissions form
9cd992a 
Ref #218

Add new annotation to get the alert banner permissions form on the manage alert banner type
entity.
Also changes the alert banner permissions generator to make sure are assigned to the alert
banner entity using BundlePermissionHandlerTrait
@andybroomfield
Copy link
Contributor Author

Done!

alexburrows pushed a commit that referenced this issue Aug 7, 2023
@andybroomfield @alexburrows
Initial support for new permissions form
0ca3b4d 
Ref #218

Add new annotation to get the alert banner permissions form on the manage alert banner type
entity.
Also changes the alert banner permissions generator to make sure are assigned to the alert
banner entity using BundlePermissionHandlerTrait
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andybroomfield