New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SendGrid] Fix recipient getting sent as reply-to address #6

Merged

rafiki270 merged 1 commit into LiveUI:master from vzsg:fix-sendgrid

Sep 7, 2018

Collaborator

vzsg commented Sep 5, 2018

As reported on Discord, trying to send an email via SendGrid throws the following error:

failure(SendGrid.SendGridError(errors: Optional([SendGrid.SendGridErrorResponse(message: Optional("The personalizations field is required and must have at least one personalization."), field: Optional("personalizations"), help: Optional("http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#-Personalizations-Errors"))])))

This PR fixes the issue by sending the recipient email address in the correct place.


          [SendGrid] Fix recipient getting sent as reply-to address

d914588

Contributor

rafiki270 commented Sep 7, 2018

Sorry for the delay, been climbing Icelandic mountains these past few days. Looks good and thanks :)

rafiki270 merged commit d9542f2 into LiveUI:master

vzsg deleted the fix-sendgrid branch

September 7, 2018 10:15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet