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

Stripe Connect Onboarding Integration #528

Merged
merged 9 commits into from
Nov 7, 2024
Merged

Conversation

Winston-Hsiao
Copy link
Contributor

@Winston-Hsiao Winston-Hsiao commented Nov 5, 2024

What does this PR do?

  1. Stripe Connected Account Onboarding (allow users to create an account or link existing stripe account)
  2. Accessible Stripe dashboard

View of account page BEFORE user has started stripe connect onboarding

Screenshot 2024-11-06 at 10 34 25 PM

View of account page AFTER a user has started stripe connect onboarding (but has not yet completed)

Screenshot 2024-11-06 at 1 18 35 AM

View of confirming details to finalize onboarding

Screenshot 2024-11-06 at 10 25 43 PM

Redirect to seller dashboard after stripe connect onboarding is completed

Screenshot 2024-11-06 at 10 26 32 PM

Account view after stripe onboarding completed

Screenshot 2024-11-06 at 10 27 35 PM

Each user gets their own stripe dashboard via their connected stripe account. They can use this to manage their products/payments.

Screenshot 2024-11-06 at 10 28 12 PM

@Winston-Hsiao Winston-Hsiao changed the title Base for stripe connect onboardnig flow Base for stripe connect onboarding flow Nov 5, 2024
@Winston-Hsiao Winston-Hsiao changed the title Base for stripe connect onboarding flow Stripe Connect Integration Nov 5, 2024
@Winston-Hsiao Winston-Hsiao self-assigned this Nov 5, 2024
@Winston-Hsiao Winston-Hsiao changed the title Stripe Connect Integration Stripe Connect Onboarding Integration Nov 6, 2024
Copy link
Member

@codekansas codekansas left a comment

Choose a reason for hiding this comment

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

nice pr - big feature once we merge it. few requests, overall structure looks great

frontend/src/components/pages/DeleteConnect.tsx Outdated Show resolved Hide resolved
frontend/src/components/pages/DeleteConnect.tsx Outdated Show resolved Hide resolved
frontend/src/components/pages/SellerDashboard.tsx Outdated Show resolved Hide resolved
frontend/src/hooks/useStripeConnect.tsx Outdated Show resolved Hide resolved
store/app/crud/users.py Outdated Show resolved Hide resolved
store/app/crud/users.py Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
Copy link
Member

@codekansas codekansas left a comment

Choose a reason for hiding this comment

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

few requests but overall greatly improved, nice job

frontend/src/components/pages/SellerOnboarding.tsx Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
store/app/routers/stripe.py Outdated Show resolved Hide resolved
@Winston-Hsiao Winston-Hsiao enabled auto-merge (squash) November 7, 2024 04:04
@Winston-Hsiao Winston-Hsiao merged commit 4352a25 into master Nov 7, 2024
1 check passed
@Winston-Hsiao Winston-Hsiao deleted the stripe-connect branch November 7, 2024 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants