- š®š¹Italy kopeboy Milan
I think we should at least have a menu item for the
assign_to_role
page, which is what one would normally delegate to non-admin users.Sidenote: if we don't add a menu item for the sibling
permission_groups
page, we should add a link to it from theassign_to_role
page.Currently the tab will be hidden because its parent /admin/people is a View page accessible only with permission 'Administer users' -- which is not necessarily what the role assigning permission groups can do. So I would make sure that even in this case, one can see the People menu item and its child Assign permission groups (it wasn't showing to me before making the View accessible).
Another general consideration and suggestion is to shorten the tab title and menu item title so that i doesn't clutter the tabs: as anticipated I would call it "Assign permission groups", without the "to roles" (which is a given in permissions).
To de-clutter the admin/people page tabs even more, we could move the
permission_groups
at<strong>Config ></strong> People > Permission Groups
, because it is creating config entities indeed, but it wouldn't be coherent with the Profiles & Roles, which are listed under People, so in the end I would leave it like it is.Thanks for the module and your great work!!
šš»