Skip to content

Conversation

@Udit-takkar
Copy link
Contributor

@Udit-takkar Udit-takkar commented Jan 20, 2026

What does this PR do?

This PR fixes the issue when you update the cal video settings on the team managed event type but the children event types were not updated.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  1. Create a manageed event type and assign some team members
  2. Update teh cal video setting on event type setup page
  3. Check if the cal video settings were updated on all the children event types.

Summary by cubic

Fixes Cal Video settings propagation for managed event types so children stay in sync. New children inherit settings; existing children are updated or cleared based on parent changes.

  • Bug Fixes
    • Create CalVideoSettings for new child event types when the parent has settings.
    • Upsert settings for existing children; delete when the parent removes Cal Video; no-op when unchanged.
    • Update handler passes the correct calVideoSettings to children based on input and Cal Video location state, with tests covering create/sync/delete/no-op.

Written for commit 78e7603. Summary will update on new commits.

@graphite-app graphite-app bot added enterprise area: enterprise, audit log, organisation, SAML, SSO core area: core, team members only labels Jan 20, 2026
@graphite-app graphite-app bot requested a review from a team January 20, 2026 10:53
@Udit-takkar Udit-takkar changed the title fix: managed children event type bug fix: managed event type bug Jan 20, 2026
@Udit-takkar Udit-takkar added the High priority Created by Linear-GitHub Sync label Jan 20, 2026
@Udit-takkar Udit-takkar added this to the v6.2 milestone Jan 20, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Copy link
Contributor

@joeauyeung joeauyeung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and working as expected

@PeerRich PeerRich merged commit 6c3b106 into main Jan 20, 2026
79 of 80 checks passed
@PeerRich PeerRich deleted the fix/children-event-type-bug branch January 20, 2026 16:29
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 High priority Created by Linear-GitHub Sync ready-for-e2e size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants