Skip to content
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.

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

Support Sendgrid Personalizations #443

Closed
cbrewster opened this issue Nov 23, 2018 · 3 comments · Fixed by #535
Closed

Support Sendgrid Personalizations #443

cbrewster opened this issue Nov 23, 2018 · 3 comments · Fixed by #535
Labels
waiting-for-info Request for more information sent, awaiting reply

Comments

@cbrewster
Copy link

cbrewster commented Nov 23, 2018

I need to be able to send out newsletters to a list of users. I would like to be able to use the same template and fill in info with substitutions on a per-user basis. Other users also should not be able to see the other recipients of the email in the to field. (See https://sendgrid.com/docs/for-developers/sending-email/personalizations/#sending-two-different-emails-to-two-different-groups-of-recipients)

From looking at the SendGrid adapter, it looks like it hardcodes a single personalization entry. It would be nice if I could add multiple personalization entries for each user and be able to send out the newsletter with a single SendGrid API call.

I am happy to actually implement this, I just want to make sure I am starting in the right direction.

@maymillerricci
Copy link
Contributor

@cbrewster Thanks for posting! This makes sense, and we would welcome a PR for it!

@lancejjohnson lancejjohnson added the waiting-for-info Request for more information sent, awaiting reply label Apr 2, 2019
@dantswain
Copy link
Contributor

I would also like this feature. I am willing to do a PR if I can get some indication that it will get attention - it's a little vague if this repo is getting active maintenance or not (see #515)

@maymillerricci
Copy link
Contributor

@dantswain We would welcome a PR for this. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-for-info Request for more information sent, awaiting reply
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants