Skip to content

Conversation

@anikdhabal
Copy link
Contributor

@anikdhabal anikdhabal commented Nov 24, 2025

What does this PR do?

Summary by cubic

Fixes loss of Google Meet link when cancelling a single attendee seat. Seat cancellations no longer modify the Google Meet conference, and video call details are preserved for other providers.

  • Bug Fixes
    • Skip updateMeeting for google_meet_video during seat cancellation to prevent Meet link removal.
    • Populate evt.videoCallData from the stored video reference (type, id, password, url) for correct updates with other providers.
    • Fetch meetingUrl, meetingId, meetingPassword, and metadata in getBookingToDelete for downstream use.

Written for commit 979c268. Summary will update automatically on new commits.

@anikdhabal anikdhabal requested a review from a team as a code owner November 24, 2025 12:52
calendarDescription: getRichDescription(evt),
};
if (reference.type.includes("_video")) {
if (reference.type.includes("_video") && reference.type !== "google_meet_video") {
Copy link
Contributor Author

@anikdhabal anikdhabal Nov 24, 2025

Choose a reason for hiding this comment

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

Skip updating video meetings for Google Meet - they're managed through Google Calendar. Currently throwing error as gmeet has no explicit videoadapter

@anikdhabal anikdhabal changed the title fix: google meet link is disappear when a seat cancelled fix: google meet link is disappear when a seat is cancelled Nov 24, 2025
@keithwillcode keithwillcode added the core area: core, team members only label Nov 24, 2025
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.

1 issue found across 2 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="packages/features/bookings/lib/handleSeats/cancel/cancelAttendeeSeat.ts">

<violation number="1" location="packages/features/bookings/lib/handleSeats/cancel/cancelAttendeeSeat.ts:97">
Video call data assignment logic duplicates `packages/features/bookings/lib/handleNewBooking/addVideoCallDataToEvent.ts:addVideoCallDataToEvent()` function. This should be refactored to use the existing utility to ensure consistency and maintainability.</violation>
</file>

Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR

Copy link
Contributor

@Udit-takkar Udit-takkar left a comment

Choose a reason for hiding this comment

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

@anikdhabal can we add some automated tests?

@github-actions
Copy link
Contributor

E2E results are ready!

@anikdhabal anikdhabal merged commit 9c2d7f9 into main Nov 25, 2025
130 of 137 checks passed
@anikdhabal anikdhabal deleted the gcal-issue branch November 25, 2025 10:55
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 ready-for-e2e size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants