-
Notifications
You must be signed in to change notification settings - Fork 555
'Delete user' - FRONT #151
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
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thank you for your submission, we really appreciate it! ❤️ Like many open-source projects, we ask that you sign our Individual Contributor License Agreement before we can accept your contribution. If you are contributing on behalf of a company, please contact us at help@plasmic.app to sign a Corporate Contributor License Agreement. You can sign the individual CLA by posting a comment with the below text. I have read, agree to, and hereby sign Plasmic's Individual Contributor License Agreement You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should also consider requiring the user to manually delete all organizations that they own first. This ensures we don't get in to the weird state where the owner of an organization is deleted, but the organization still has members.
@@ -609,6 +609,10 @@ export abstract class SharedApi { | |||
return this.post("/auth/self/password", { oldPassword, newPassword }); | |||
} | |||
|
|||
async deactivateUser(email: string): Promise<{}> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move this after sendEmailVerification
key="delete" | ||
onClick={async () => { | ||
const confirm = await reactConfirm({ | ||
title: `Delete user account`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete your account?
} | ||
}} | ||
> | ||
<strong>Delete</strong> user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete account
onClick={async () => { | ||
const confirm = await reactConfirm({ | ||
title: `Delete user account`, | ||
message: <>Are you sure you want to delete your account?</>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure you want to delete your account with email ${email}?
You will immediately lose access to all data you own, including your projects, CMS databases, workspaces, and organizations. Your account and data will be permanently deleted within 30 days.
Add 'Delete user' function :