search

ualibraries / Guide-on-the-Side

The University of Arizona Libraries will no longer provide support for Guide on the Side. The code will remain openly available; however, UAL can no longer provide code fixes or upgrades.
https://ualibraries.github.io/Guide-on-the-Side/about.html
Other
66 stars 48 forks source link

Print / Send email fails with multiple emails and whitespace #153

Closed caosborne89 closed 7 years ago

caosborne89 commented 7 years ago

Perhaps this issue has already been considered, but a consistent issue seems to be that when someone puts in multiple emails and there is white space between the emails in additional to the commas, the email won't send nor is there any error message that comes up.

caosborne89 commented 7 years ago

If anyone has time to confirm this, that would be much appreciated. To test this I put 3 emails (working ones) into the "Email address(es):" section like this: "email1@email.com , email2@email.com , email3.email.com" With a space or two in between the emails and the commas. screenshot_gots

caosborne89 commented 7 years ago

Oops I must of been on an old branch when I went to test this. I actually fixed this a while ago ref #145. Seems to be working for me.