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

Me: presence of premium themes should not prevent closure of account #27014

Closed
bluefuton opened this issue Sep 5, 2018 · 17 comments · Fixed by #27411
Closed

Me: presence of premium themes should not prevent closure of account #27014

bluefuton opened this issue Sep 5, 2018 · 17 comments · Fixed by #27411
Assignees
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Privacy/GDPR [Type] Bug

Comments

@bluefuton
Copy link
Contributor

Steps to reproduce

Using an account containing a site with a premium theme:

  1. Starting at URL: https://wordpress.com/me/account/close

What I expected

It is possible to delete the account.

What happened instead

screen shot 2018-09-05 at 12 18 25

The copy instructs the user to cancel active purchases before deleting the account, but it is not possible to cancel a premium theme (it's a one-time purchase).

Context / Source

Reported by a user in 1392647-zen.

When checking whether an account has active purchases, we should probably ignore premium themes.

cc @westi

@bluefuton bluefuton added [Type] Bug [Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. labels Sep 5, 2018
@bluefuton bluefuton added this to the Account Closure milestone Sep 5, 2018
@KokkieH
Copy link
Contributor

KokkieH commented Sep 5, 2018

Also related to #17012

@KokkieH
Copy link
Contributor

KokkieH commented Sep 24, 2018

Another case from the forums

@KokkieH
Copy link
Contributor

KokkieH commented Jul 8, 2019

@bluefuton it looks like there might be a regression here: In https://en.forums.wordpress.com/topic/delete-my-account-permanently-10/ a user reported being unable to close their account. After an HE removed the premium theme (bought in 2015, so definitely non-refundable), they were able to close the account.

I've seen several users contact us in the forums the past two weeks about being unable to close their accounts, but this is the first case I've been able to confirm that it works after the theme is removed.

@KokkieH KokkieH reopened this Jul 8, 2019
@KokkieH
Copy link
Contributor

KokkieH commented Jul 8, 2019

I'm able to replicate the issue, using the same testing steps as I'd used when reviewing #27411:

  • Created a new account and site
  • Purchase a Premium Theme using credits so it's non-refundable
  • Confirm theme appears on Purchases page with no option to cancel
  • Attempt to close account

Where normally on an account that owns upgrades you get the prompt to contact support the moment you click on the Close Account option, I made it through the entire flow, including the confirmation step, before getting an error:

Account_Settings_—_WordPress_com

Only after manually removing the premium theme from the account using internal tools, I was able to close the account.

@darnelldibbles-zz
Copy link

@darnelldibbles-zz
Copy link

@darnelldibbles-zz
Copy link

@KokkieH
Copy link
Contributor

KokkieH commented Jul 29, 2019

Another in https://en.forums.wordpress.com/topic/closing-account-144/

Of note is they were apparently able to delete the site with the premium theme attached with no problem, but still got an error when trying to close the account.

@bluefuton
Copy link
Contributor Author

Looks like the problem is on the API side (that message is shown when the deletion API request fails).

@KokkieH are you able to inspect the API response using the 'Network' tab when attempting to delete with your test user above?

@KokkieH
Copy link
Contributor

KokkieH commented Aug 6, 2019

I'm seeing this response:

{"code":403,"headers":[{"name":"Content-Type","value":"application\/json"}],"body":{"error":"active-subscriptions","message":"This user account cannot be closed while it has active subscriptions."}}

@bluefuton
Copy link
Contributor Author

Thanks for testing that @KokkieH 👍

It looks like Calypso is applying the right logic, but the API endpoint doesn't know to exclude premium themes from the 'active subscriptions' check.

@KokkieH
Copy link
Contributor

KokkieH commented Aug 7, 2019

Noting here as an additional scenario to test for a fix:

I just had a case in https://en.forums.wordpress.com/topic/delete-account-2201 where someone had a premium theme subscription on a site that itself had been successfully deleted three weeks ago. So the site deletion with the active theme subscription worked, but closing the account still doesn't work.

@bluefuton
Copy link
Contributor Author

Working on an API fix in D31970-code.

@KokkieH
Copy link
Contributor

KokkieH commented Sep 27, 2019

Noting I came across a test account with a non-refundable premium theme I'd forgotten about, and when I tried to close the account it worked without any issues, so it looks like this might be working again. Test account username premtestk

@bluefuton
Copy link
Contributor Author

I've just deployed D31970-code in r201400-wpcom. This ignores non-refundable premium themes at the endpoint when the active purchases check is made ✅

@KokkieH
Copy link
Contributor

KokkieH commented Jan 9, 2020

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Privacy/GDPR [Type] Bug
Projects
None yet
4 participants