Skip to content

fix: invalid cal video bug#13058

Merged
PeerRich merged 4 commits intomainfrom
fix/invalid-cal-video
Jan 8, 2024
Merged

fix: invalid cal video bug#13058
PeerRich merged 4 commits intomainfrom
fix/invalid-cal-video

Conversation

@Udit-takkar
Copy link
Copy Markdown
Contributor

@Udit-takkar Udit-takkar commented Jan 5, 2024

What does this PR do?

Fixes #12971
Fixes #13060

To reproduce the bug

  1. Create an event type that requires confirmation with cal video as location
  2. Book a meeting and then accept the meeting from /bookings/unconfirmed
  3. Reschedule the meeting on Incognito tab or another browser
  4. confirm the meeting /bookings/unconfirmed and go to the meeting url

you would get the error "room does not exist or not found" because the room has been deleted when you rescheduled.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 5, 2024

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

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 5, 2024 5:50pm
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 5, 2024 5:50pm
5 Ignored Deployments
Name Status Preview Comments Updated (UTC)
ai ⬜️ Ignored (Inspect) Visit Preview Jan 5, 2024 5:50pm
cal ⬜️ Ignored (Inspect) Visit Preview Jan 5, 2024 5:50pm
cal-demo ⬜️ Ignored (Inspect) Jan 5, 2024 5:50pm
qa ⬜️ Ignored (Inspect) Visit Preview Jan 5, 2024 5:50pm
ui ⬜️ Ignored (Inspect) Visit Preview Jan 5, 2024 5:50pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@github-actions github-actions Bot added the High priority Created by Linear-GitHub Sync label Jan 5, 2024
@keithwillcode keithwillcode added the core area: core, team members only label Jan 5, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

);
}

const DisplayLocation = ({
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just created a common component

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

smart

Comment on lines +337 to +338
const latestVideoReference = videoReferences[videoReferences.length - 1];

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

use the latest video reference instead of the first one

@pumfleet
Copy link
Copy Markdown
Contributor

pumfleet commented Jan 6, 2024

Thank you for fixing this, I ran into this issue a few times and it got super annoying

@PeerRich
Copy link
Copy Markdown
Member

PeerRich commented Jan 7, 2024

can we have E2E tests for this?

@PeerRich PeerRich merged commit 020515d into main Jan 8, 2024
@PeerRich PeerRich deleted the fix/invalid-cal-video branch January 8, 2024 13:10
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 High priority Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-2877] Invalid Cal Video link on calendar event [CAL-2850] After reschedule Cal Video link is broken

4 participants