Skip to content
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: Make id field required for CalendarCache #20579

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

hariombalhara
Copy link
Member

@hariombalhara hariombalhara commented Apr 7, 2025

Make id field required in CalendarCache.

We could do it along with #18619 but it is better to do it earlier because

  • It keeps the PR 18619 a bit smaller
  • If there are any issues in deployment due to this migration, we detect them before and fix them quicker

@github-actions github-actions bot added the ❗️ migrations contains migration files label Apr 7, 2025
@keithwillcode keithwillcode added core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO labels Apr 7, 2025
Copy link

vercel bot commented Apr 7, 2025

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Apr 9, 2025 9:46am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Apr 9, 2025 9:46am

@hariombalhara hariombalhara marked this pull request as ready for review April 9, 2025 09:34
@hariombalhara hariombalhara requested a review from a team as a code owner April 9, 2025 09:34
@graphite-app graphite-app bot requested a review from a team April 9, 2025 09:34
Copy link

graphite-app bot commented Apr 9, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (04/09/25)

1 reviewer was added to this PR based on Keith Williams's automation.

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 enterprise area: enterprise, audit log, organisation, SAML, SSO ❗️ migrations contains migration files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants