anymail / django-anymail

Django email backends and webhooks for Amazon SES, Brevo (Sendinblue), MailerSend, Mailgun, Mailjet, Postmark, Postal, Resend, SendGrid, SparkPost, Unisender Go and more
https://anymail.dev
BSD 3-Clause "New" or "Revised" License
1.7k stars 130 forks source link

Sendgrid multiple reply to #325

Closed gdvalderrama closed 1 year ago

gdvalderrama commented 1 year ago

which ESP you're using (Mailgun, SendGrid, etc.)

Sendgrid

what versions of Anymail, Django, and Python you're running

Anymail v10.0 Django v3.2 Python v3.9

any error messages and exception stack traces

AnymailUnsupportedFeature SendGrid does not support multiple reply_to addresses


It looks like Sengrid supports multiple "reply to"s (see changelog).

However, Anymail still documents a single reply_to as a Sendgrid limitation, see docs.

It looks like this is still limited by Anymail here, but when having multiple "reply_to"s, the reply_to_list should be used instead.

medmunds commented 1 year ago

Thanks for reporting this and providing the pointers to SendGrid's docs.

You've correctly identified exactly where to fix it. Want to open a PR to switch Anymail from reply_to to reply_to_list?