Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Design concept for deleting users (or organisations) #211

Open
tillmo opened this issue Sep 25, 2017 · 3 comments
Open

Design concept for deleting users (or organisations) #211

tillmo opened this issue Sep 25, 2017 · 3 comments

Comments

@tillmo
Copy link
Member

tillmo commented Sep 25, 2017

Should deletion be possible at all? If yes, should all repos deleted as well, or should the user remain in the database and just be deactivated? If the former, then how to handle dangling references into deleted repos?

@eugenk
Copy link
Member

eugenk commented Nov 16, 2017

I think deletion should be possible. I suggest:

  • If a user wants to delete their own account, there must not be any repositories left in their namespace. Not even the ones that are scheduled for deletion.
  • Any admin of a repository can schedule a deletion of the repository.
  • When an admin of a repository schedules a deletion, the repository is marked scheduled_for_deletion for a day and is not listed any more in the API (except for its admins). All admins of the repository are notified via email about the deletion. Only admins of the repository can see it and unschedule the deletion. The actual deletion happens asynchronously after a day.

@phyrog
Copy link
Contributor

phyrog commented Nov 20, 2017

Github also has the restriction that a user cannot be deleted if the user is an admin of an organization. We could also restrict this restriction to the last admin.

@tillmo
Copy link
Member Author

tillmo commented Nov 20, 2017

Also, the last admin should not be able to degrade him/herself to a non-admin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants