feat: cancel subscription #159
Conversation
|
@Ipriyankrajai is attempting to deploy a commit to the Inbox Zero Team on Vercel. A member of the Team first needs to authorize it. |
apps/web/utils/actions.ts
Outdated
| @@ -106,6 +125,9 @@ export async function deleteAccountAction() { | |||
| captureException(error); | |||
| } | |||
|
|
|||
| if (lemonSqueezySubscriptionItemId) | |||
| await cancelSubScriptionForUser(lemonSqueezySubscriptionItemId); | |||
|
|
|||
| await prisma.user.delete({ where: { email: session.user.email } }); | |||
There was a problem hiding this comment.
I think we should delete the user after the subscription tenure is ended.
There was a problem hiding this comment.
No, don't delete the user. "leaving a user's data in Stripe after they delete their account is generally allowed under GDPR, provided that you have a legitimate reason for retaining this data, such as for financial records, compliance with tax law, or other regulatory requirements."
There was a problem hiding this comment.
For now i have commented that part
apps/web/utils/actions.ts
Outdated
| @@ -106,6 +125,9 @@ export async function deleteAccountAction() { | |||
| captureException(error); | |||
| } | |||
|
|
|||
| if (lemonSqueezySubscriptionItemId) | |||
| await cancelSubScriptionForUser(lemonSqueezySubscriptionItemId); | |||
|
|
|||
| await prisma.user.delete({ where: { email: session.user.email } }); | |||
There was a problem hiding this comment.
No, don't delete the user. "leaving a user's data in Stripe after they delete their account is generally allowed under GDPR, provided that you have a legitimate reason for retaining this data, such as for financial records, compliance with tax law, or other regulatory requirements."
| id: true, | ||
| tier: true, |
There was a problem hiding this comment.
We don't need id or tier here, right?
| if (lemonSqueezySubscriptionId) | ||
| await cancelSubScriptionForUser(lemonSqueezySubscriptionId); | ||
|
|
||
| // await prisma.user.delete({ where: { email: session.user.email } }); |
There was a problem hiding this comment.
Uncomment. We do want to delete the user.
(We do want to keep the user in Lemon though)
fixes: #153