-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
fix: scheduleEmailReminders cron job #11929
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes! |
}, | ||
}, | ||
}); | ||
skip: pageNumber * pageSize, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
load workflowReminders in batches (we also do that in api/crondowngradeUsers
and api/cron/monthlyDigestEmail
)
📦 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✅ 162 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 10/17/2023 07:01:06am UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: dceb828 Started: 10/17/2023 06:57:56am UTC
|
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Teams Profile page is loaded for users in Organization
Retry 2 • Retry 1 • Initial Attempt |
1.49% (4)4 / 269 runsfailed over last 7 days |
30.48% (82)82 / 269 runsflaked over last 7 days |
📄 apps/web/playwright/change-username.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Change username on settings User can change username
Retry 1 • Initial Attempt |
1.44% (4)4 / 278 runsfailed over last 7 days |
25.18% (70)70 / 278 runsflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/action-based.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 |
---|---|---|
Popup Tests should be able to reschedule
Retry 1 • Initial Attempt |
9.03% (26)26 / 288 runsfailed over last 7 days |
89.58% (258)258 / 288 runsflaked over last 7 days |
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1 • Initial Attempt |
3.81% (11)11 / 289 runsfailed over last 7 days |
35.64% (103)103 / 289 runsflaked over last 7 days |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find @CarinaWolli
LGTM
What does this PR do?
Attempt to fix failing github action
Cron-scheduleEmailReminders
(https://github.com/calcom/cal.com/actions/workflows/cron-scheduleEmailReminders.yml) --> currently disabledAxiom error:
Type of change