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 Integration: checkout and order pages #240

Merged
merged 6 commits into from
Aug 26, 2024

Conversation

annstriganova
Copy link
Collaborator

@annstriganova annstriganova commented Jun 6, 2024

Stripe integration for frontend part.

Test in conjunction with local BE instance, refer to Local Frontend + Backend instruction.

Test cards list: https://docs.stripe.com/checkout/embedded/quickstart?client=react#testing

@annstriganova
Copy link
Collaborator Author

annstriganova commented Aug 1, 2024

Scenario 1: no address is specified

no_address.mov

Scenario 2: address is specified
NOTE the issue with the items counter UPD: fixed
out

@annstriganova annstriganova marked this pull request as ready for review August 5, 2024 14:24
@annstriganova annstriganova changed the title Stripe PoC: checkout and order stubs Stripe Integration: checkout and order pages Aug 5, 2024
Copy link

@horprogs horprogs left a comment

Choose a reason for hiding this comment

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

Т.к. вы используете TypeScript, то нужно задавать типы в коде

Dockerfile.local Outdated Show resolved Hide resolved
src/components/Cart/CartFull/CartFull.tsx Outdated Show resolved Hide resolved
src/components/Orders/Orders.tsx Outdated Show resolved Hide resolved
@annstriganova
Copy link
Collaborator Author

@horprogs, спасибо! 🚀

@Sunagatov Sunagatov merged commit 606b26c into development Aug 26, 2024
12 of 13 checks passed
@Sunagatov Sunagatov deleted the feature/stripe-integration branch August 26, 2024 20:22
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.

None yet

3 participants