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

Resolved[#8732] Adding Trim to remove extra space

Open pranjalg13 opened this issue 2 years ago • 4 comments

What does this PR do?

Added .trim() to the email value while adding new member so that it is not treated as a new email.

Fixes #8732

Environment: Staging(main branch) / Production

Type of change

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

How should this be tested?

  • [x] Without the changes the space between email while adding a new member was considered as a new member
  • [x] Now with the changes it wont be (added .trim to the value_

pranjalg13 avatar May 07 '23 13:05 pranjalg13

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2023 7:37pm

vercel[bot] avatar May 07 '23 13:05 vercel[bot]

@pranjalg13 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 May 07 '23 13:05 vercel[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 May 07 '23 13:05 github-actions[bot]

@zomars please review this once

pranjalg13 avatar May 07 '23 13:05 pranjalg13

@zomars please review this once. Thanks

pranjalg13 avatar May 10 '23 05:05 pranjalg13