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

feat(auth): allow confirming account deletion #201

Open
insertish opened this issue May 19, 2023 · 2 comments · May be fixed by #321
Open

feat(auth): allow confirming account deletion #201

insertish opened this issue May 19, 2023 · 2 comments · May be fixed by #321
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@insertish
Copy link
Member

Route needs to be added to auth routes.

https://github.com/revoltchat/revite/blob/d55d86aa3cb5cd0b5b29646b499dac5b16f20c1e/src/pages/login/ConfirmDelete.tsx#L16
@insertish insertish added help wanted Extra attention is needed good first issue Good for newcomers labels May 19, 2023
@insertish insertish added this to the Settings milestone May 19, 2023
@insertish insertish removed this from the Settings milestone Mar 28, 2024
@sabs21
Copy link

sabs21 commented Sep 1, 2024

By confirming account deletion, do you mean something more than what's currently being done when you click 'Delete Account' in the user settings? It seems like confirmation exists already
image

@insertish
Copy link
Member Author

The /login/... route that takes a token and confirms the account should be deleted.
(from the email sent to user when they request deletion)

@Drevoed Drevoed linked a pull request Oct 10, 2024 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
Revolt Project
Status: 🏗 In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(auth): Add confirm delete page Drevoed/frontend
2 participants
@sabs21 @insertish