-
Notifications
You must be signed in to change notification settings - Fork 129
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
Allow a user to delete their own account #2122
base: master
Are you sure you want to change the base?
Allow a user to delete their own account #2122
Conversation
peregrineshahin
commented
Jul 20, 2024
e97d8cc
to
5428051
Compare
How about flagging the account as deleted, instead of deleting it from the database? |
5428051
to
9ca8e2e
Compare
Tests will not be deleted and shall be accessible with links as they are not user-url-encoded, although the test page of the user will not show their tests as it will give 404 after clicking their tests link from a test which is fine I think. |
After the account is deleted, is it still possible (e.g. by somebody else) to reregister the same account name? If so, this certainly does not feel right to me and means that there should be at least a list of past account names kept. But you might as well "flag" the account as deleted then (along with removing all possible personal data except the user name), as @vdbergh alludes to. |
That's not the only way to prevent impersonation, we can check for registeration on demand if there was any action related such username before. |
9ca8e2e
to
b8f264f
Compare
b8f264f
to
fac91f2
Compare
some comments here that come to my mind... nothing definitive.
|