Vijay
Vijay
@hbjORbj , Gentle reminder! 🙏
@hbjORbj , reminding 🙏
> Hey @vijayraghav-io, I noticed there are quite a few unrelated changes in the PR (things like added whitespace, new lines, reorganising of imports, and minor formatting like extra commas...
@kart1ka , cleaned up the diff.
Reminding...
> I did notice, In the second test case you are using the `newGoogleCalCredentialId`, This might be involuntary as `google_meet_video` should ideally connect to the `newGoogleMeetCredentialId` instead of `newGoogleCalCredentialId`. Good...
> Would be better, if you have test case for this functionality. @vijayraghav-io can you please add ? sure will add
@Praashh , added an e2e test
No, this https://github.com/calcom/cal.com/pull/17443 is not reconnecting booking references, which is the purpose of this current PR
Verified multiple times locally the new test is passing, along with bookings-list.e2e module. Will figure out and update to remove flakiness on server.