-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
chore: add DKIM error handling email to cal.ai #12902
Conversation
@DexterStorey is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
📦 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! 🙌 |
This PR is being marked as stale due to inactivity. |
Hey there, there is a merge conflict, can you take a look? |
This PR is being marked as stale due to inactivity. |
@DexterStorey Is this something we should still do? |
Hi @keithwillcode I don't think this is necessary. If there are elevated issues resulting from DKIM we can re-open, but this seems to be rare! Cheers, |
What does this PR do?
Handles dkim verification failure with a specific email to the user.
Fixes # (issue)
Requirement/Documentation
Type of change
How should this be tested?
Mandatory Tasks
Checklist
I haven't read the contributing guideMy code doesn't follow the style guidelines of this projectI haven't commented my code, particularly in hard-to-understand areasI haven't checked if my PR needs changes to the documentationI haven't checked if my changes generate no new warningsI haven't added tests that prove my fix is effective or that my feature worksI haven't checked if new and existing unit tests pass locally with my changes