Skip to content

feat: add delete account button to settings page#4644

Closed
GMetaxakis wants to merge 2 commits intoONEARMY:masterfrom
GMetaxakis:feat/delete-profile
Closed

feat: add delete account button to settings page#4644
GMetaxakis wants to merge 2 commits intoONEARMY:masterfrom
GMetaxakis:feat/delete-profile

Conversation

@GMetaxakis
Copy link
Copy Markdown
Contributor

@GMetaxakis GMetaxakis commented Mar 1, 2026

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

PR Type

  • Feature

What is the new behavior?

  • Replaces the Discord link placeholder on the Account Settings page with a real account deletion form
  • Users enter their password for confirmation before deletion
  • Account is deleted via the Supabase admin API, which cascades to clean up related data (profile, map pins, messages, badges, tags, subscribers)
  • Authored content is preserved but de-attributed (created_by set to NULL)
  • On success, the user is redirected to the homepage with auth state cleared

Does this PR introduce a DB Schema Change or Migration?

  • No

Git Issues

Closes #4163

@GMetaxakis
feat: add delete account button to settings page
3fb61b3 
Replace the Discord link placeholder with a real account deletion form.
Users enter their password for confirmation, and the account is deleted
via the Supabase admin API which cascades to clean up related data.

Closes ONEARMY#4163
@cypress
Copy link
Copy Markdown

cypress bot commented Mar 1, 2026 •
edited
Loading

onearmy-community-platform    Run #8746

Run Properties:  status check failed Failed #8746  •  git commit cf3ddb00b8: test: add e2e tests for account deletion
Project onearmy-community-platform
Branch Review pull/4644
Run status status check failed Failed #8746
Run duration 24m 40s
Commit git commit cf3ddb00b8: test: add e2e tests for account deletion
Committer Georgios Metaxakis
View all properties for this run ↗︎
Test results
Tests that failed  Failures 3
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 87
View all changes introduced in this branch ↗︎

Tests for review

Failed  research/write.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[Research] > [Create research article] > [By Authenticated] Test Replay Screenshots Video
Failed  SignUp.spec.ts • 2 failed tests • ci-chrome

View Output Video

Test Artifacts
[User sign-up] > [Update existing auth details] > Updates username and password Test Replay Screenshots Video
[User sign-up] > [Delete account] > Deletes account with correct password Test Replay Screenshots Video
Flakiness  src/integration/research/write.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Research] > [Displays draft updates for Author] > [By Authenticated] Test Replay Screenshots Video

@GMetaxakis GMetaxakis marked this pull request as ready for review March 1, 2026 18:33
@GMetaxakis GMetaxakis requested a review from a team as a code owner March 1, 2026 18:33
@GMetaxakis
Copy link
Copy Markdown
Contributor Author

GMetaxakis commented Mar 5, 2026

Closing in favour of #4650 which has a cleaner implementation (accordion + confirmation modal). E2e tests have been moved there.

@GMetaxakis GMetaxakis closed this Mar 5, 2026
@GMetaxakis GMetaxakis deleted the feat/delete-profile branch March 5, 2026 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

Community Platform
Status: Done
Core Team - BAU list
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[feature] Add Profile delete button

1 participant

@GMetaxakis