Unknown admin on teams #169

Closed
opened 2 years ago by mpeltriaux · 2 comments
Owner

Status quo

Regular users can see who is part of their team but there is not information on who is currently administrator.

Improvement

Add info about current admin to team overview.

# Status quo Regular users can see who is part of their team but there is not information on who is currently administrator. # Improvement Add info about current admin to team overview.
mpeltriaux added the
enhancement
label 2 years ago
mpeltriaux self-assigned this 2 years ago
mpeltriaux referenced this issue from a commit 2 years ago
mpeltriaux referenced this issue from a commit 2 years ago
mpeltriaux referenced this issue from a commit 2 years ago
Poster
Owner

Further improvements

  • refactored single-admin to multi-admins on Team
  • adds tests for user app
  • adds safer delete for Teams
    • since shared access can be managed using Teams it would be dangerous to allow users to simply delete Teams from the database. Worst case: They do not understand the consequences of this action, despite the warning explanation on the delete-confirm form.
    • this way the Teams are being 'deleted' the same ways major datatypes are being 'deleted' (setting a deleted UserAction entry on the database)
    • the 'restore deleted data' has been ported to Team admin backend as well to restore 'deleted' Teams in case of need
# Further improvements * refactored single-admin to multi-admins on Team * adds tests for user app * adds safer delete for Teams * since shared access can be managed using Teams it would be dangerous to allow users to simply delete Teams from the database. Worst case: They do not understand the consequences of this action, despite the warning explanation on the delete-confirm form. * this way the Teams are being 'deleted' the same ways major datatypes are being 'deleted' (setting a deleted UserAction entry on the database) * the 'restore deleted data' has been ported to Team admin backend as well to restore 'deleted' Teams in case of need
mpeltriaux referenced this issue from a commit 2 years ago
Poster
Owner

Merged in #170

# Merged in #170
mpeltriaux closed this issue 2 years ago
Sign in to join this conversation.
Loading…
There is no content yet.