-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
fix/stripe-customer-sync #3153
fix/stripe-customer-sync #3153
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
}); | ||
|
||
// Notify stripe about the change | ||
if (updatedUser && updatedUser.metadata && hasKeyInMetadata(updatedUser, "stripeCustomerId")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zomars Should I add the same flag to know if we are on self hosted? And don't execute any stripe code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some self-hosted users will use Stripe Connect, If this is only for premium usernames then we can skip it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me get this right, connect can be for self hosted users, but this it's only involves our user plan handling so I should add flag here for IS_SELF_HOSTED
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nevermind seems like is good to go 👍🏽
What does this PR do?
Takes care of keeping in sync user data with stripe customer data
Fixes # (issue)
Environment: Staging(main branch) / Production
Type of change
How should this be tested?
Checklist