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 Brevo batch sending with merge_data #353

Closed
waldeck-dev opened this issue Jan 23, 2024 · 2 comments · Fixed by #357
Closed

Support Brevo batch sending with merge_data #353

waldeck-dev opened this issue Jan 23, 2024 · 2 comments · Fixed by #357

Comments

@waldeck-dev
Copy link

Hi,

I'm reaching out to report a potential issue I encountered while reading the Anymail docs (for use with Brevo). This is my first time working with both Anymail and Brevo, so I might be getting it all wrong.

The documentation states (here) that Brevo does not support per-recipient variables, and we shouldn't use merge_data.

From what I understood from the Brevo documentation (here), it looks like per-recipient params are supported (using params in each subobjects of messageVersions).

As of my writing, this section of the Brevo documentation was updated one month ago. Is this a feature that was recently supported, or am I misunderstanding something?

@medmunds
Copy link
Contributor

It looks like Brevo recently added batch sending support (via a new messageVersions field in their sending API). Thanks for pointing it out.

We'll need to update Anymail's Brevo backend to add support for this. Anymail's merge_data will flow into Brevo's per-recipient params, as you've noted. I'm not sure whether Anymail's merge_metadata (per-recipient metadata) can be supported. Metadata uses Brevo's X-Mailin-custom email header, and Brevo's docs don't indicate that headers can be customized per recipient in messageVersions. (But it's worth testing—docs are often incomplete.)

@medmunds medmunds changed the title About Brevo and batch sending/merge Support Brevo batch sending with merge_data Jan 23, 2024
medmunds added a commit that referenced this issue Feb 19, 2024
medmunds added a commit that referenced this issue Feb 19, 2024
medmunds added a commit that referenced this issue Feb 19, 2024
@medmunds
Copy link
Contributor

(Released in Anymail 10.3)

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

Successfully merging a pull request may close this issue.

2 participants