Skip to content

fix: Stripe integration#9462

Merged
PeerRich merged 15 commits into
calcom:mainfrom
zodman:FIX_stripe_integration
Jul 1, 2023
Merged

fix: Stripe integration#9462
PeerRich merged 15 commits into
calcom:mainfrom
zodman:FIX_stripe_integration

Conversation

@zodman
Copy link
Copy Markdown
Contributor

@zodman zodman commented Jun 12, 2023

What does this PR do?

Fixes #9365

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

automated test

/claim #9365

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api 🔄 Building (Inspect) Jun 30, 2023 4:13am
cal 🔄 Building (Inspect) Jun 30, 2023 4:13am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2023 4:13am
web-staging 🔄 Building (Inspect) Jun 30, 2023 4:13am

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 12, 2023

@zodman is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 12, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@PeerRich PeerRich added the Medium priority Created by Linear-GitHub Sync label Jun 17, 2023
@keithwillcode keithwillcode requested a review from a team June 18, 2023 19:17
@keithwillcode
Copy link
Copy Markdown
Contributor

@zodman Thanks for your contribution. For future PRs, can you please fill in the PR description? 🙏

@keithwillcode keithwillcode changed the title Stripe integration working again fix: Stripe integration Jun 18, 2023
@zodman
Copy link
Copy Markdown
Contributor Author

zodman commented Jun 20, 2023

@keithwillcode this PR is complement of #9365 I need to know if this needs to be finish for finish #9365

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2023

Thank you for following the naming conventions! 🙏

@PeerRich PeerRich requested a review from a team June 25, 2023 15:14
@PeerRich PeerRich added the automated-tests area: unit tests, e2e tests, playwright label Jun 25, 2023
Copy link
Copy Markdown
Contributor

@alannnc alannnc left a comment

Choose a reason for hiding this comment

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

Code looks good and it's working.
@roae Is this exactly what we wanted?

@PeerRich PeerRich merged commit 0e022b0 into calcom:main Jul 1, 2023
fritterhoff pushed a commit to hm-edu/cal.com that referenced this pull request Jul 25, 2023
* Add Stripe integration tests

* Add Stripe payment feature

* Update Stripe payment form with valid card info

* Fill postal code if visible in Stripe test

* Implementing new e2e test

* lint fix

---------

Co-authored-by: Peer Richelsen <peeroke@gmail.com>
Co-authored-by: alannnc <alannnc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-tests area: unit tests, e2e tests, playwright Medium priority Created by Linear-GitHub Sync

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

[CAL-1879] Make E2E test: Pending payment booking should not be confirmed by default

4 participants