rdmorganiser / rdmo

A tool to support the planning, implementation, and organization of research data management.
https://rdmorganiser.github.io
Apache License 2.0
105 stars 49 forks source link

In Admin-Ansicht: "Mitarbeiter-Status" in "Manager-Status" ändern #511

Open Zack-83 opened 2 years ago

Zack-83 commented 2 years ago

grafik

Rationale / Begründung

Wenn ich die Unterscheidung richtig verstehe, können Nutzer mit "mittleren Rechten" Kataloge ändern und die Schaltfläche "Management" betreiben, aber nicht Kataloge löschen und die Schaltfläche "Admin" betreiben (das können nur diejenigen mit den höchsten Rechten).

Minimal functionality

Ich würde "Mitarbeiter" in "Manager" editieren. Das ist kohärenter mit der Benennung der Schaltflächen und mit der vermutlichen Rolle der Person im echten Leben (FDM-Manager oder FDM-Koordinator).

Die Rollenbeschreibung könnte auch verbessert werden:

Aktiv Legt fest, ob dieser Benutzer aktiv ist. Kann deaktiviert werden, anstatt das Konto zu löschen. Manager Legt fest, ob dieser Benutzer Inhalte (Fragenkataloge, Ansichte, usw.) editieren kann. Administrator Legt fest, ob dieser Benutzer alle Berechtigungen hat (Inhalte löschen, Rollen zuweisen, usw.), ohne diese einzeln zuweisen zu müssen.

Active Designates whether this user should be treated as active. Unselect this instead of deleting accounts. Manager Designates whether this user can edit contents (catalogues, views, etc.). Administrator Designates whether this user has all permissions (delete contents, assign roles, etc.) without explicitly assigning them.

Nice-to-have functionality

Wenn inaktiv, könnte ggf. die Schaltfläche "Admin" ausgeblendet werden.

jochenklar commented 2 years ago

Hi @Zack-83 , das ist noch ein bisschen anders, der "Mitarbeiter" status ist nichts anderes als das man sich in das Django Admin Interface einloggen kann. Normalerweise sollte das in RDMO nicht benötigt werden da "Manager" die Management Interface nutzen können.

Wenn man nicht-Admins in das Admin Interface lässt, dann kann man ihnen Rechte und/oder Gruppen zuordnen (feingranular per Model). Ich würde das aber eigentlich eher lassen.

MyPyDavid commented 2 years ago

ein Nutzer in der Editor Gruppe ist ein "Manager" und sieht die Management Interface? ein Nutzer mit der Rolle Manager für die Site (site_managers) sieht alle Projekte der Site?

ich wäre auch vor eine kleine Umbenennung ;)

jochenklar commented 2 years ago

Ja, nur können wir das in Django nicht machen. Wir könnten das Management Interface in Content umbennen, aber das finde ich auch suboptimal. Ist ja quasi auf allen scrennshots drauf.