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.

Sign up for GitHub

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

Jump to bottom

Allow to disable users first before deleting them #31

Closed
tcitworld opened this issue Sep 15, 2019 · 3 comments · Fixed by #519
Closed

Allow to disable users first before deleting them #31

tcitworld opened this issue Sep 15, 2019 · 3 comments · Fixed by #519
Assignees
@tcitworld
Labels
enhancement good first issue

Comments

@tcitworld
Copy link
Member

Since deleting users is a rather destructive action, it would be nice to first disable them before actually deleting them just in case.

Needs a new input field to set the number of days after which an user should be disabled.
@tcitworld tcitworld self-assigned this Sep 15, 2019
@coffeemesh
Copy link

I really would appreciate this feature! Is anyone actively working on this?
Otherwise I will be looking into it and hopefully figure out how to implement it myself, any and every help would be appreciated.

@nickvergessen
Copy link
Member

I guess the code place is around
https://github.com/nextcloud/user_retention/blob/master/lib/BackgroundJob/ExpireUsers.php#L117

You can take inspiration from the files retention:
https://github.com/nextcloud/files_retention/blob/master/lib/BackgroundJob/RetentionJob.php#L173-L175

@tcitworld
Copy link
Member Author

I had started to write an app for this a while ago, feel free to have a look https://framagit.org/framasoft/nextcloud/purge

@nickvergessen nickvergessen mentioned this issue Oct 24, 2022
Closed
mejo- added a commit that referenced this issue Feb 24, 2023
@mejo-
Allow to disable users instead of deleting them.
c11eb16 
Add new config parameters for disabling normal and guest users and
allow to configure them in the settings UI.

Fixes: #31

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- mentioned this issue Feb 24, 2023
Merged
nickvergessen pushed a commit that referenced this issue Feb 27, 2023
@mejo- @nickvergessen
Allow to disable users instead of deleting them.
226eb33 
Add new config parameters for disabling normal and guest users and
allow to configure them in the settings UI.

Fixes: #31

Signed-off-by: Jonas <jonas@freesources.org>
nickvergessen pushed a commit that referenced this issue Mar 20, 2023
@mejo- @nickvergessen
Allow to disable users instead of deleting them.
83d9794 
Add new config parameters for disabling normal and guest users and
allow to configure them in the settings UI.

Fixes: #31

Signed-off-by: Jonas <jonas@freesources.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tcitworld tcitworld

Labels
enhancement good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow to disable users instead of deleting them. nextcloud/user_retention
3 participants
@nickvergessen @tcitworld @coffeemesh