cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

fix: Allow impersonation from the /settings/admin/users page

Open anikdhabal opened this issue 1 year ago β€’ 5 comments

Fixes #13240 /claim #13240

Screenshot 2024-01-17 205916

anikdhabal avatar Jan 17 '24 15:01 anikdhabal

@anikdhabal is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jan 17 '24 15:01 vercel[bot]

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

github-actions[bot] avatar Jan 17 '24 15:01 github-actions[bot]

πŸ“¦ 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! πŸ™Œ

github-actions[bot] avatar Jan 17 '24 15:01 github-actions[bot]

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

Hybes avatar Jan 18 '24 10:01 Hybes

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

anikdhabal avatar Jan 18 '24 14:01 anikdhabal

https://github.com/calcom/cal.com/assets/81948346/2b11f7d1-d4f5-416a-9586-06c46fa246c4

But it's working fine for me @Udit-takkar.

anikdhabal avatar Jan 19 '24 15:01 anikdhabal

@anikdhabal could you also add the modal like in the /teams? I re review this again or ask someone else to confirm

Udit-takkar avatar Jan 19 '24 15:01 Udit-takkar

@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.

anikdhabal avatar Jan 19 '24 15:01 anikdhabal

https://github.com/calcom/cal.com/assets/81948346/811aaf39-d393-4bd5-91bc-2fa8b902968a

@Udit-takkar

anikdhabal avatar Jan 19 '24 16:01 anikdhabal