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

fix: prevent duplicate calendar account linking

Open Amit91848 opened this issue 1 year ago • 3 comments

What does this PR do?

Fixes #13302

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

Mandatory Tasks

  • [x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Amit91848 avatar Jan 18 '24 18:01 Amit91848

@Amit91848 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 18 '24 18: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 18 '24 18: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 18 '24 18:01 github-actions[bot]

Hey @Amit91848 I wonder if there is a way to make this into a utility to prevent code duplication; also wondering if the check can be done before credential creation as this is undone later.

emrysal avatar Jan 22 '24 12:01 emrysal

@Udit-takkar for lark if user tries to sign in using the same account, new credentials are overwriting the older ones. Should I change this? Will check for others.

Amit91848 avatar Jan 22 '24 15:01 Amit91848

@emrysal made changes.

Amit91848 avatar Jan 22 '24 16:01 Amit91848

Hey there, there is a merge conflict, can you take a look?

github-actions[bot] avatar Jan 29 '24 20:01 github-actions[bot]