pacnpal/django-anymail
1
0
Fork 0
mirror of https://github.com/pacnpal/django-anymail.git synced 2025-12-20 03:41:05 -05:00
Code Issues Packages Projects Releases Wiki Activity

SendGrid: Fix multiple recipients with only merge_global_data

Browse Source 
In SendGrid backend, support non-batch template send to multiple
recipients when `merge_global_data` is set without `merge_data`.
Regression introduced in v6.0.

Fixes #179
This commit is contained in:
Mike Edmunds
2020-03-18 17:08:10 -04:00
committed by GitHub
parent 4245d468ec
commit 920d8dd70f
3 changed files with 30 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/test_sendgrid_backend.py
View File

@@ -493,6 +493,20 @@ class SendGridBackendAnymailFeatureTests(SendGridBackendMockAPITestCase):
'custom_args': {'anymail_id': 'mocked-uuid-2'},
'substitutions': {"<%test%>": "data"}}])
def test_merge_data_global_only(self):
# a template with only global data can be used to send the same message
# to multiple recipients (non-batch)
self.message.template_id = "d-5a963add2ec84305813ff860db277d7a"
self.message.merge_global_data = {"test": "data"}
self.message.to = ["one@example.com", "two@example.com"]
self.message.send()
data = self.get_api_call_json()
self.assertEqual(data['personalizations'], [
{'to': [{'email': 'one@example.com'}, {'email': 'two@example.com'}], # not batch
'custom_args': {'anymail_id': 'mocked-uuid-1'},
'dynamic_template_data': {"test": "data"}}])
def test_legacy_merge_data(self):
# unless a new "dynamic template" is specified, Anymail assumes the legacy
# "substitutions" format for merge data