Results 37 comments of Mehul

While updating the PR, I noticed that it is possible to book a event with *length* of 0. This is possible when saving the form from a tab other than...

All those form tabs are implemented as a single form. It would be weird to save multiple times when you are switching between tabs and configuring settings.

![Explaination](https://github.com/calcom/cal.com/assets/86384652/360390a4-c46d-4d80-8a94-9be274001eb8) > we should not be showing "Nameless" anywhere in the UI. > > i know we do it somewhere when there is no username but ideally we get rid...

![Explaination](https://github.com/calcom/cal.com/assets/86384652/ec8c534e-8d00-410e-9f25-157de7ea2e73) E2E 3 are having issue, because the seats are being created without the responses basic values E2E 7,9,10 are passing on my side, dont know why they are failing...

> If that's intended, can you fix the tests so they pass? > > Could have been flaky tests. We are not running e2e tests on every PR anymore, but...

While trying to fix types i accidentally broke the functionality. Reverting it back for the time being.

> While trying to fix types i accidentally broke the functionality. Reverting it back for the time being. resolved Hey @joeauyeung, It's ready for review.

> @MehulZR looks like there are conflicts Thanks for the heads-up, I am working on an another PR, will work on this one soon.

![Explaination ](https://github.com/calcom/cal.com/assets/86384652/97188915-e944-4535-86d4-cff7bb68a3dc) I can start working on this but there is slight confusion. According to https://github.com/calcom/cal.com/issues/8809#issue-1703499760, do we want to provide the connectedEmails as an option or not?

https://github.com/calcom/cal.com/blob/851cd4d9938a979241d5fcc74691d4c0c96ad5f5/packages/core/getBusyTimes.ts#L157-L158 I have added some tests & in those tests the value of seatReferences returned is undefined. Causing the tests to fail. It probably has something to do with prisma...