Skip to content

refactor: app-store imports webhook.ts from @calcom/web#13257

Merged
PeerRich merged 1 commit into
mainfrom
fix-app-store-imports-web
Jan 16, 2024
Merged

refactor: app-store imports webhook.ts from @calcom/web#13257
PeerRich merged 1 commit into
mainfrom
fix-app-store-imports-web

Conversation

@ThyMinimalDev
Copy link
Copy Markdown
Contributor

@ThyMinimalDev ThyMinimalDev commented Jan 16, 2024

What does this PR do?

Fixes app-store importing file from web for no reason

Type of change

  • Chore (refactoring code, technical debt, workflow improvements)

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@ThyMinimalDev ThyMinimalDev changed the title refactor: app-store import webhook.ts from @calcom/web refactor: app-store imports webhook.ts from @calcom/web Jan 16, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 16, 2024

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@keithwillcode keithwillcode added the core area: core, team members only label Jan 16, 2024
@ThyMinimalDev
Copy link
Copy Markdown
Contributor Author

this file "cal.com/packages/app-store/paypal/api/index.ts" imports webhooks file from @calcom/web, but the file in @calcom/web just exports the same webhook file by importing it from app-store

same thing with alby app

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 16, 2024

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

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2024 2:45pm
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2024 2:45pm
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2024 2:45pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 16, 2024 2:45pm
cal-demo ⬜️ Ignored (Inspect) Jan 16, 2024 2:45pm
ui ⬜️ Ignored (Inspect) Visit Preview Jan 16, 2024 2:45pm

@keithwillcode keithwillcode added foundation High priority Created by Linear-GitHub Sync labels Jan 16, 2024
@keithwillcode keithwillcode added this to the v3.8 milestone Jan 16, 2024
@PeerRich PeerRich enabled auto-merge (squash) January 16, 2024 14:37
@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jan 16, 2024

Current Playwright Test Results Summary

✅ 380 Passing - ⚠️ 16 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 01/16/2024 03:09:09pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: e961f49

Started: 01/16/2024 02:59:12pm UTC

⚠️ Flakes

📄   apps/web/playwright/booking-seats.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Reschedule for booking with seats If rescheduled/cancelled booking with seats it should display the correct number of seats
Retry 1Initial Attempt
1.96% (7) 7 / 357 runs
failed over last 7 days
0.56% (2) 2 / 357 runs
flaked over last 7 days

📄   apps/web/playwright/booking/longTextQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Long Text Question and Each Other Question Long Text required and Number not required
Retry 1Initial Attempt
0.27% (1) 1 / 367 run
failed over last 7 days
2.18% (8) 8 / 367 runs
flaked over last 7 days

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to enable 2FA and login using 2FA
Retry 2Retry 1Initial Attempt
0% (0) 0 / 272 runs
failed over last 7 days
35.66% (97) 97 / 272 runs
flaked over last 7 days

📄   apps/web/playwright/booking/selectQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Phone Question and Each Other Question Booking With Select Question and Address Question Select and Address not required
Retry 1Initial Attempt
0.27% (1) 1 / 365 run
failed over last 7 days
1.64% (6) 6 / 365 runs
flaked over last 7 days

📄   apps/web/playwright/booking/radioGroupQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Radio Question and Each Other Question Booking With Radio Question and Address Question Booking With Radio Question and Phone Question Radio and Phone not required
Retry 1Initial Attempt
0.27% (1) 1 / 367 run
failed over last 7 days
1.91% (7) 7 / 367 runs
flaked over last 7 days

📄   apps/web/playwright/booking/checkboxGroupQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Checkbox Group Question and Each Other Question Booking With Checkbox Group Question and Address Question Booking With Checkbox Group Question and Radio group Question Checkbox Group and Radio group not required
Retry 1Initial Attempt
0% (0) 0 / 370 runs
failed over last 7 days
1.08% (4) 4 / 370 runs
flaked over last 7 days

📄   apps/web/playwright/booking/availability.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Check availability tab in a event-type Check availability in event type
Retry 1Initial Attempt
26.19% (11) 11 / 42 runs
failed over last 7 days
30.95% (13) 13 / 42 runs
flaked over last 7 days

📄   apps/web/playwright/event-types.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Event Types tests user -- legacy Different Locations Tests Can add Cal video location and book with it
Retry 1Initial Attempt
0.26% (1) 1 / 383 run
failed over last 7 days
3.66% (14) 14 / 383 runs
flaked over last 7 days

📄   apps/web/playwright/organization/organization-invitation.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Organization Email not matching orgAutoAcceptEmail Team invitation
Retry 1Initial Attempt
5.60% (22) 22 / 393 runs
failed over last 7 days
27.23% (107) 107 / 393 runs
flaked over last 7 days

📄   apps/web/playwright/booking/phoneQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Phone Question and Each Other Question Booking With Phone Question and Address Question Booking With Phone Question and checkbox group Question Phone and checkbox group required
Retry 1Initial Attempt
1.31% (5) 5 / 382 runs
failed over last 7 days
2.36% (9) 9 / 382 runs
flaked over last 7 days

📄   apps/web/playwright/oauth-provider.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
OAuth Provider should create valid access token & refresh token for team
Retry 1Initial Attempt
0% (0) 0 / 400 runs
failed over last 7 days
18.50% (74) 74 / 400 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 4 Flakes

Top 1 Common Error Messages

null

4 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 2Retry 1Initial Attempt
13.42% (53) 53 / 395 runs
failed over last 7 days
82.03% (324) 324 / 395 runs
flaked over last 7 days
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1Initial Attempt
2.23% (9) 9 / 404 runs
failed over last 7 days
54.21% (219) 219 / 404 runs
flaked over last 7 days
Popup Tests should open Routing Forms embed on click
Retry 1Initial Attempt
-2.03% (-8) -8 / 394 runs
failed over last 7 days
43.65% (172) 172 / 394 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe according to system theme when no theme is configured through Embed API
Retry 1Initial Attempt
2.79% (11) 11 / 394 runs
failed over last 7 days
76.65% (302) 302 / 394 runs
flaked over last 7 days

📄   apps/web/playwright/availability.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Availablity tests Date Overrides
Retry 1Initial Attempt
0.75% (3) 3 / 402 runs
failed over last 7 days
2.74% (11) 11 / 402 runs
flaked over last 7 days

View Detailed Build Results


@PeerRich PeerRich merged commit ba944b4 into main Jan 16, 2024
@PeerRich PeerRich deleted the fix-app-store-imports-web branch January 16, 2024 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only foundation High priority Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants