search

zorn-v / nextcloud-social-login

GNU Affero General Public License v3.0
199 stars 138 forks source link

Group Mapping matching UI #423

Open kousu opened 1 year ago

kousu commented 1 year ago

The current Group Mapping UI allows reusing NextCloud groups. I think this is probably undesirable? Once a group is chosen it should be removed from the options so it can't accidentally be reused.

If you intentionally want subgroups, you should set it up on the source identity server to keep things consistent.

This probably requires redesigning the UI of Add Group Mapping component to use a different kind of widget.

zorn-v commented 1 year ago

I think this is probably undesirable?

Why not ? What if you can not manage groups on identity server ?