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: stripe connect integration for payouts #2041

Merged
merged 29 commits into from
Apr 29, 2024
Merged

Conversation

kashif-m
Copy link
Contributor

@kashif-m kashif-m commented Aug 29, 2023

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

This includes integration of StripeConnect APIs in Stripe connector for payouts functionality. For doing the same, there is an addition of a new Payout connector action (PayoutRecipientAccountCreation). This helps in creation of external accounts (bank / card) for Stripe Connect users.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

Stripe Connect is a virtual marketplace for onboarding and paying out to platform's affiliates / vendors / resellers. Integrating Stripe Connect offers payout abstraction for such use-cases.

How did you test it?

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

@kashif-m kashif-m requested review from a team as code owners August 29, 2023 10:12
@github-actions github-actions bot added the S-conventions-not-followed Status: This PR does not follow contributing guidelines label Aug 29, 2023
@kashif-m kashif-m self-assigned this Aug 29, 2023
@kashif-m kashif-m changed the title Payout stripe connect feat: stripe connect integration for payouts Aug 29, 2023
@kashif-m kashif-m added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed and removed S-conventions-not-followed Status: This PR does not follow contributing guidelines labels Aug 29, 2023
@SamraatBansal SamraatBansal added the M-configuration-changes Metadata: This PR involves configuration changes label Aug 30, 2023
@kashif-m kashif-m requested a review from a team as a code owner August 31, 2023 09:24
crates/api_models/src/payouts.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe.rs Show resolved Hide resolved
crates/router/src/connector/stripe.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/connector/stripe/transformers.rs Outdated Show resolved Hide resolved
crates/router/src/core/utils.rs Outdated Show resolved Hide resolved
@kashif-m kashif-m force-pushed the payout_stripe_connect branch 5 times, most recently from a993881 to fa712a0 Compare October 13, 2023 10:58
ArjunKarthik
ArjunKarthik previously approved these changes Apr 26, 2024
Copy link
Contributor

@ArjunKarthik ArjunKarthik left a comment

Choose a reason for hiding this comment

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

Approving just to remove my previous requested changes. Please get it reviewed by the respective folks

@ArjunKarthik ArjunKarthik requested review from ArjunKarthik and removed request for ArjunKarthik April 26, 2024 12:22
Copy link
Contributor

@ArjunKarthik ArjunKarthik left a comment

Choose a reason for hiding this comment

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

Approving just to remove my previous requested changes. Please get it reviewed by the respective folks

@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Apr 29, 2024
Merged via the queue into main with commit ac9d856 Apr 29, 2024
10 of 12 checks passed
@Gnanasundari24 Gnanasundari24 deleted the payout_stripe_connect branch April 29, 2024 10:13
@pixincreate pixincreate removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-configuration-changes Metadata: This PR involves configuration changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Integrate Stripe Connect for paying out