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

Mark User as deleted before deleting in X weeks completely #19553

Open
6543 opened this issue Apr 29, 2022 · 1 comment
Open

Mark User as deleted before deleting in X weeks completely #19553

6543 opened this issue Apr 29, 2022 · 1 comment
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active. type/proposal The new feature has not been accepted yet but needs to be discussed first.

Comments

@6543
Copy link
Member

6543 commented Apr 29, 2022

"Wouldn't it be better to have the API mark the user "to be deleted" and delete after X weeks ? This would make revert easier, in case someone account is compromised."

good idea, will require extra work for ALL delete and GET functions from models, so i thin it's a own issue (and worth it's own pull)

Originally posted by @6543 in #19439 (comment)
@6543 6543 added type/proposal The new feature has not been accepted yet but needs to be discussed first. type/feature Completely new functionality. Can only be merged if feature freeze is not active. labels Apr 29, 2022
@jolheiser jolheiser changed the title Mark User as deleted bevor deleting in X weeeks completly Mark User as deleted before deleting in X weeks completely Apr 29, 2022
@lunny
Copy link
Member

lunny commented May 9, 2022

I think we can use the exist prohibit status to stop user do anything and then have a cron to delete it in background.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active. type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lunny @6543