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.

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

Account deactivation happens without confirmation #16870

Closed
Animajosser opened this issue Apr 6, 2021 · 6 comments
Closed

Account deactivation happens without confirmation #16870

Animajosser opened this issue Apr 6, 2021 · 6 comments
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression X-Spec-Changes Z-Spec-Compliance An area where Element doesn't correctly implement the spec Z-Synapse

Comments

@Animajosser
Copy link

Description

I am pretty sad, because I lost my account with this issue.

I wanted to look at the section detailing account deactivation, so I clicked it. It was really stupid in hindsight, as I expected a confirmation dialog and wasn't thinking of clicking confirm. However the software decided I agreed and the dialog disappeared as soon as it appeared, without me doing anything, and my account was deactivated. I was left with the settings screen still visible and the background changing to white. When clicking behind the settings screen I could create a new account.

I tried with a dummy account, I still had time to click on a checkbox, this also immediately led to the same result, without any confirmation from my side.

Steps to reproduce

  • create account.
  • click on deactivate account in settings.
  • don't click on confirm (I couldn't even see the button at all. it went really fast for me)
  • account will be gone.

Describe how what happens differs from what you expected.

I expected a confirmation dialog waiting for me to click it. I would click: "NO" and it would be over.

Version information

webapp in Firefox 87.0 on Linux.

@SimonBrandner SimonBrandner added S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect labels Apr 6, 2021
@erikjohnston
Copy link
Member

matrix-org/synapse#9754 is the corresponding Synapse issue

@turt2live turt2live added X-Regression X-Spec-Changes Z-Spec-Compliance An area where Element doesn't correctly implement the spec and removed T-Defect labels Apr 6, 2021
@turt2live
Copy link
Member

I've flagged this as "Spec Compliance" which isn't super accurate but the best we have for "we made a wrong assumption and the spec didn't help us"

@BrenBarn
Copy link

This is not priority "Major". This is "Critical". This is the kind of bug that will turn people away from matrix for the rest of their lives.

@ShadowJonathan
Copy link
Contributor

ShadowJonathan commented Jun 16, 2021

A quick hack to put 1 more element-local modal over that button would work in this case.

@clokep
Copy link

clokep commented Jun 16, 2021

Synapse v1.37.0 will remove this behavior, see matrix-org/synapse#10184.

@turt2live
Copy link
Member

Considering fixed for now due to that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression X-Spec-Changes Z-Spec-Compliance An area where Element doesn't correctly implement the spec Z-Synapse
Projects
None yet
Development

No branches or pull requests

8 participants