search

Kuestenschmiede / LdapBundle

This con4gis bundle adds LDAP functionality for Contao backend and frontend users/groups.
https://con4gis.org
GNU Lesser General Public License v3.0
1 stars 5 forks source link

Allow multiple administrator groups #12

Closed dorianim closed 2 years ago

dorianim commented 2 years ago

Hey there,

It would be nice to have an option to define multiple administrator groups. We currently get this warning Screenshot_2022-03-22-10-45-29_1920x1080 stating that at least one group, which is not an admin group, has access to the template editor and therefore has implicit admin access. If it was possible to define multiple admin groups, we could avoid this warning.

coastforge-cro commented 2 years ago

We will make changes so that multiple admin groups can be selected.

But I wonder: why don't you merge the two groups, which you want to be admin groups, together? Since both groups should be admin groups, both groups would have the same permissions anyway and there would be no difference between the two groups, therefore they could be merged. Or am I missing something?

dorianim commented 2 years ago

Thanks!

why don't you merge the two groups

One of these groups is for "global admins", they have admin access in every tool we use. The other group is only for "website admins", they only have admin access in Contao. These are two different groups of people, so we can't really merge them.

coastforge-cro commented 2 years ago

One of these groups is for "global admins", they have admin access in every tool we use. The other group is only for "website admins", they only have admin access in Contao. These are two different groups of people, so we can't really merge them.

Ah, I see. Makes perfect sense then.

I will report back here as soon as I have finished creating a solution.

coastforge-mei commented 2 years ago

The change has been implemented for a while. Only today we were able to make the version.