fix: Removing payment fee on stripe#10652
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Ignored Deployments
|
|
Thank you for following the naming conventions! 🙏 |
| </div> | ||
| </> | ||
| }> | ||
| }}> |
There was a problem hiding this comment.
I tried to find a link on stripe connect fees, but they are specific for each country.
| stripeAccount: this.credentials.stripe_user_id, | ||
| }) as unknown as Prisma.InputJsonValue, | ||
| fee: paymentFee, | ||
| fee: 0, |
There was a problem hiding this comment.
fee is required when creating payment.
|
🤖 Meticulous spotted visual differences in 59 of 171 screens tested: view and approve differences detected. Last updated for commit 1dc38e8. This comment will update as new commits are pushed. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Current Playwright Test Results Summary✅ 116 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 08/18/2023 02:31:58am UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: 1dc38e8 Started: 08/18/2023 02:30:56am UTC
|
| Test Case | Last 7 days Failures | Last 7 days Flakes |
|---|---|---|
|
Popup Tests should be able to reschedule
Retry 1 • Initial Attempt |
1.91% (5)5 / 262 runsfailed over last 7 days |
98.09% (257)257 / 262 runsflaked over last 7 days |
📄 apps/web/playwright/webhook.e2e.ts • 1 Flake
Test Case Results
| Test Case | Last 7 days Failures | Last 7 days Flakes |
|---|---|---|
|
FORM_SUBMITTED can submit a form and get a submission event
Retry 1 • Initial Attempt |
2.65% (7)7 / 264 runsfailed over last 7 days |
25% (66)66 / 264 runsflaked over last 7 days |
|
when are we planning to reintroduce it? |
We already talked about something, at the end of the month calculate total % of booking paid and then charge the user with that fee. |
This reverts commit 7a7a5b5.
What does this PR do?
Fixes #10650
Requirement/Documentation
Type of change
How should this be tested?
Mandatory Tasks
Checklist