Anik Dhabal Babu

Results 34 comments of Anik Dhabal Babu

> @anikdhabal there are some merge conflicts Fixed it.

> Looks like this one consistently fails tests So what am I going to do now? Also, those tests are not related to the changes.

> @anikdhabal No, no worries. We'll look into getting the tests fixed. Thank you.

> this test "login flow user & logout using dashboard" is only failing in this PR. I see two sidebar when i run this E2E test locally on this branch....

Hey @Udit-takkar @keithwillcode two Layout is visible in future version. ![Screenshot 2024-02-08 002649](https://github.com/calcom/cal.com/assets/81948346/969686f8-b7a7-4b22-8e90-3938c37a0ccd) ![Screenshot 2024-02-08 002631](https://github.com/calcom/cal.com/assets/81948346/6009aa8f-591f-4151-a56a-0c07e38adad0)

![Screenshot 2024-02-08 010822](https://github.com/calcom/cal.com/assets/81948346/5b5b7378-7f7d-40b8-b3fc-927fedef9f87) The reason for having two layouts in future mode is that we wrap the component with getLayout. Therefore, we either have to ensure that all components in...

I've fixed the double layout issue.

@keithwillcode finally. Thanks for your review.

> ## To fix > When day e.g. "Friday" is toggled 2 confirmation messages appear instead of 1. Loom: https://www.loom.com/share/627cca63e7f44adcb3e32d63eacdbb2c > > > ## UX thoughts > 1. Should we...

> Great work! Should other references of WEBAPP_URL also be refactored in this PR itself or a separate one? No, will handle it in this Pr.