fix: Restore Google Meet videoCallUrl in metadata#28689
fix: Restore Google Meet videoCallUrl in metadata#28689Sumanthvu wants to merge 4 commits intocalcom:mainfrom
Conversation
|
It looks like this PR includes formatting changes that are not aligned with the repository configuration. The project uses Biome (see biome.json), which explicitly enforces:
However, several changes in this PR remove semicolons and adjust formatting inconsistently with Biome rules. This suggests the code may have been formatted with a different tool (e.g. Prettier with semi: false) or without running the project's formatter. To keep the codebase consistent and avoid unnecessary diff noise, it would be better to:
This PR should ideally contain only the functional fix. |
francescogatto
left a comment
There was a problem hiding this comment.
Too much noise, please commit only relevant fix without any change to other changes to other files
Ryukemeister
left a comment
There was a problem hiding this comment.
hi there, thank you for your contribution. @francescogatto is right here, this is too much noise. can you also commit the relevant files and revert the other please. can you also attach a visual demo of the fix before/after, thank you!
What does this PR do?
This PR resolves the regression where the Google Meet
videoCallUrlwas missing from the booking metadata.Following the Calendar Sync refactor, the code was incorrectly assuming the calendar integration payload would always be located at
results[0]. This PR updatesRegularBookingService.tsto dynamically extract thehangoutLinkfrom the actual calendar result rather than relying on a hardcoded array index. It also adds regression tests tofresh-booking.test.tsto ensure the URL is properly persisted.