cal.com
cal.com copied to clipboard
fix: Allow impersonation from the /settings/admin/users page
Fixes #13240 /claim #13240
@anikdhabal is attempting to deploy a commit to the cal Team on Vercel.
A member of the Team first needs to authorize it.
Thank you for following the naming conventions! π Feel free to join our discord and post your PR link.
π¦ 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! π
hi @Udit-takkar and @anikdhabal, the functionality works, but when you click the impersonate button, it routes you to: /settings/admin/users as the new user. Which they would not have permission to. also, I don't think the VenetianMask is the standard Icon to use for Impersonation
hi @Udit-takkar and @anikdhabal, the functionality works, but when you click the impersonate button, it routes you to: /settings/admin/users as the new user. Which they would not have permission to. also, I don't think the VenetianMask is the standard Icon to use for Impersonation
Thanks for the review. I've fixed that. @Udit-takkar @PeerRich
https://github.com/calcom/cal.com/assets/81948346/2b11f7d1-d4f5-416a-9586-06c46fa246c4
But it's working fine for me @Udit-takkar.
@anikdhabal could you also add the modal like in the /teams? I re review this again or ask someone else to confirm
@anikdhabal could you also add the modal like in the /teams? I re review this again or ask someone else to confirm
Ok Let me quickly add that.
https://github.com/calcom/cal.com/assets/81948346/811aaf39-d393-4bd5-91bc-2fa8b902968a
@Udit-takkar