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
refactor: Add crm
type to app credentials
#14147
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @joeauyeung and the rest of your teammates on Graphite |
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
crm
type to app credentials
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 4 Ignored Deployments
|
📦 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✅ 301 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 04/09/2024 04:45:06pm UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: 58a0cb7 Started: 04/09/2024 04:41:22pm UTC
|
|
2 Test Cases Affected |
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Insights should be able to go to insights as admins
Retry 1 • Initial Attempt |
0% (0)0 / 316 runsfailed over last 7 days |
4.11% (13)13 / 316 runsflaked over last 7 days |
Insights should be able to go to insights as members
Retry 1 • Initial Attempt |
0% (0)0 / 316 runsfailed over last 7 days |
2.22% (7)7 / 316 runsflaked over last 7 days |
📄 apps/web/playwright/profile.e2e.ts • 2 Flakes
Top 1 Common Error Messages
|
2 Test Cases Affected |
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Update Profile Can add a new email as a secondary email
Retry 1 • Initial Attempt |
0.31% (1)1 / 326 runfailed over last 7 days |
0.61% (2)2 / 326 runsflaked over last 7 days |
Update Profile Can resend verification link if the secondary email is unverified
Retry 1 • Initial Attempt |
4.01% (13)13 / 324 runsfailed over last 7 days |
14.20% (46)46 / 324 runsflaked 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 matching orgAutoAcceptEmail and a Verified Organization with DNS Setup Done nonexisting user is invited to a team inside organization
Retry 1 • Initial Attempt |
1.18% (3)3 / 255 runsfailed over last 7 days |
10.20% (26)26 / 255 runsflaked over last 7 days |
📄 apps/web/playwright/integrations-stripe.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Stripe integration Pending payment booking should not be confirmed by default
Retry 2 • Retry 1 • Initial Attempt |
1.26% (4)4 / 317 runsfailed over last 7 days |
25.24% (80)80 / 317 runsflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 8 Flakes
Top 1 Common Error Messages
|
8 Test Cases Affected |
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1 • Initial Attempt |
0.31% (1)1 / 319 runfailed over last 7 days |
61.13% (195)195 / 319 runsflaked over last 7 days |
Popup Tests should be able to reschedule
Retry 1 • Initial Attempt |
-157.72% (-194)-194 / 123 runsfailed over last 7 days |
157.72% (194)194 / 123 runsflaked over last 7 days |
Popup Tests should open Routing Forms embed on click
Retry 1 • Initial Attempt |
-156.91% (-193)-193 / 123 runsfailed over last 7 days |
156.91% (193)193 / 123 runsflaked 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 1 • Initial Attempt |
-156.10% (-192)-192 / 123 runsfailed over last 7 days |
156.91% (193)193 / 123 runsflaked 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 configured with 'auto' theme using Embed API
Retry 1 • Initial Attempt |
-158.20% (-193)-193 / 122 runsfailed over last 7 days |
158.20% (193)193 / 122 runsflaked over last 7 days |
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Booker Profile Page) with dark theme when configured with dark theme using Embed API
Retry 1 • Initial Attempt |
-158.20% (-193)-193 / 122 runsfailed over last 7 days |
158.20% (193)193 / 122 runsflaked over last 7 days |
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Event Booking Page) with dark theme when configured with dark theme using Embed API
Retry 1 • Initial Attempt |
-158.20% (-193)-193 / 122 runsfailed over last 7 days |
158.20% (193)193 / 122 runsflaked over last 7 days |
Popup Tests prendered embed should be loaded and apply the config given to it
Retry 1 • Initial Attempt |
-158.20% (-193)-193 / 122 runsfailed over last 7 days |
158.20% (193)193 / 122 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 on submitting team form, triggers team webhook
Retry 1 • Initial Attempt |
0.62% (2)2 / 323 runsfailed over last 7 days |
24.46% (79)79 / 323 runsflaked over last 7 days |
📄 apps/web/playwright/teams.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Teams - NonOrg -- future Team Onboarding Invite Members
Retry 1 • Initial Attempt |
7.95% (26)26 / 327 runsfailed over last 7 days |
22.63% (74)74 / 327 runsflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/preview.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Preview Preview - embed-core should load
Retry 1 • Initial Attempt |
0% (0)0 / 320 runsfailed over last 7 days |
33.44% (107)107 / 320 runsflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/inline.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Inline Iframe Inline Iframe - Configured with Dark Theme
Retry 1 • Initial Attempt |
0.63% (2)2 / 320 runsfailed over last 7 days |
52.19% (167)167 / 320 runsflaked over last 7 days |
Graphite Automations"Add foundation team as reviewer" took an action on this PR • (04/05/24)1 reviewer was added to this PR based on Keith Williams's automation. |
LGTM again - just re-running CI as e2e failing. Doesnt seem relevant to this PR , they may be flaky |
@sean-brydon @joeauyeung Code LGTM. But how are we going to update the App records in the DB without a proper migration? |
In the next PR in this stack I made it so credentials with "_other_calendar" are still treated as CRM apps. What are your thoughts? Should we still write a DB migration? |
What does this PR do?
This is a part of our bigger CRM refactor.
Currently, our CRM apps have the type of
_other_calendar
which gets processed with our other calendar apps. This PR aims to introduce the_crm
type to credentials in preparation for a CRM manager. Since this is a bigger change I wanted it to be it's own PR to make reviewing easier.Changes
other_calendar
Fixes # (issue)
Requirement/Documentation
Type of change
How should this be tested?
Mandatory Tasks
Checklist