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

feat(app): stripe app validate customer data #2158

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

hekike
Copy link
Contributor

@hekike hekike commented Jan 27, 2025

Validate if the Stripe app customer and payment method exist before saving it.

@hekike hekike added release-note/feature Release note: Exciting New Features area/billing labels Jan 27, 2025
@hekike hekike requested review from solkimicreb and turip January 27, 2025 19:55
@hekike hekike changed the title Feat/stripe app validate customer data feat(app): stripe app validate customer data Jan 27, 2025
@@ -582,24 +582,24 @@ func (a adapter) GetMaskedSecretAPIKey(secretAPIKeyID secretentity.SecretID) (st
}

// getStripeAppClient returns a Stripe App Client based on App ID
func (a adapter) getStripeAppClient(ctx context.Context, appID appentitybase.AppID) (stripeclient.StripeAppClient, error) {
func (a adapter) getStripeAppClient(ctx context.Context, appID appentitybase.AppID) (appstripeentity.AppData, stripeclient.StripeAppClient, error) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit maybe return a single type instead of returning three values

@hekike hekike merged commit 7f7d26c into main Jan 27, 2025
26 checks passed
@hekike hekike deleted the feat/stripe-app-validate-customer-data branch January 27, 2025 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/billing release-note/feature Release note: Exciting New Features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants