search

axisj / react-multi-email

:octocat: A simple react component to format multiple email as the user types.
https://react-multi-email.vercel.app/
MIT License
298 stars 72 forks source link

fix(reactmultiemail.tsx): trim emails with space after comma #165

Closed jhaltercmu2020 closed 1 year ago

jhaltercmu2020 commented 1 year ago

trim emails with space after comma. If you copy something like: email1@domain.com, email2@domain.com, email3@domain.com the space is

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
react-multi-email ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 5:26pm
jhaltercmu2020 commented 1 year ago

@thomasJang Can you take a look? The reason why we need to make this change is because if you copy something like this:

email1@domain.com, email2@domain.com, email3@domain.com

the spaces are causing issues and if we trim the space everything works as expected. Let me know if you have any questions.

thomasJang commented 1 year ago

Yes!, I think this code don't make side effect.

jhaltercmu2020 commented 1 year ago

@thomasJang Thanks, would you be able to tag a few others with approval access to review?

thomasJang commented 1 year ago

Thx. @jhaltercmu2020 1.0.18 is out