Send one mail instead of n mail to supporter on checkout of n grants

app/grants/sync/helpers.py

@@ -75,7 +75,11 @@ def record_contribution_activity(contribution):
         # successful_contribution(grant, subscription, contribution)
         # update_grant_metadata.delay(grant.pk)
         new_supporter(grant, subscription)
-        thank_you_for_supporting(grant, subscription)
+        grants_with_subscription = [{
+            'grant': grant,
+            'subscription': subscription
+        }]
+        thank_you_for_supporting(grants_with_subscription)
 
     except Exception as e:
         logger.error(f"error in record_contribution_activity: {e} - {contribution}")

app/grants/tasks.py

@@ -13,7 +13,9 @@
 from celery.utils.log import get_task_logger
 from dashboard.models import Profile
 from grants.models import Grant, Subscription
-from marketing.mails import new_grant, new_grant_admin, new_supporter, thank_you_for_supporting
+from marketing.mails import (
+    new_grant, new_grant_admin, new_supporter, thank_you_for_supporting
+)
 from marketing.models import Stat
 from perftools.models import JSONStore
 from townsquare.models import Comment
@@ -159,13 +161,14 @@ def update_grant_metadata(self, grant_id, retry: bool = True) -> None:
 
 
 @app.shared_task(bind=True, max_retries=1)
-def process_grant_contribution(self, grant_id, grant_slug, profile_id, package, retry: bool = True) -> None:
+def process_grant_contribution(self, grant_id, grant_slug, profile_id, package, send_supporter_mail:bool = True, retry: bool = True):
     """
     :param self:
     :param grant_id:
     :param grant_slug:
     :param profile_id:
     :param package:
+    :param send_supporter_mail:
     :return:
     """
     from grants.views import record_subscription_activity_helper
@@ -252,8 +255,38 @@ def process_grant_contribution(self, grant_id, grant_slug, profile_id, package,
         new_supporter(grant, subscription)
 
         # emails to contributor
-        thank_you_for_supporting(grant, subscription)
+        if send_supporter_mail:
+            grants_with_subscription = [{
+                'grant': grant,
+                'subscription': subscription
+            }]
+            thank_you_for_supporting(grants_with_subscription)
+
         update_grant_metadata.delay(grant_id)
+        return grant, subscription
+
+
+@app.shared_task(bind=True, max_retries=1)
+def batch_process_grant_contributions(self, grants_with_payload, profile_id, retry: bool = True) -> None:
+    """
+    :param self:
+    :param grants_with_payload: list of dicts with grant_id, grant_slug and payload
+    :param profile_id:
+    :return:
+    """
+    grants_with_subscription = []
+    for grant_with_payload in grants_with_payload:
+        grant_id = grant_with_payload["grant_id"]
+        grant_slug = grant_with_payload["grant_slug"]
+        payload = grant_with_payload["payload"]
+        grant, subscription = process_grant_contribution(
+            grant_id, grant_slug, profile_id, payload, send_supporter_mail=False, retry=retry
+        )
+        grants_with_subscription.append({
+            "grant": grant,
+            "subscription": subscription
+        })
+    thank_you_for_supporting(grants_with_subscription)
 
 
 @app.shared_task(bind=True, max_retries=1)

app/grants/views.py

@@ -2057,8 +2057,9 @@ def bulk_fund(request):
     successes = []
     failures = []
 
-    batch_grants_mail = []
     profile = get_profile(request)
+    grants_with_payload = []
+
     for (index, grant_id) in enumerate(grant_ids_list):
         try:
             grant = Grant.objects.get(pk=grant_id)
@@ -2103,7 +2104,6 @@ def bulk_fund(request):
         )
 
         try:
-            from grants.tasks import process_grant_contribution
             payload = {
                 # Values that are constant for all donations
                 'checkout_type': request.POST.get('checkout_type'),
@@ -2139,7 +2139,11 @@ def bulk_fund(request):
                 'token_symbol': request.POST.get('token_symbol').split(',')[index],
                 'include_for_clr': json.loads(request.POST.get('include_for_clr', 'true'))
             }
-            process_grant_contribution.delay(grant_id, grant.slug, profile.pk, payload)
+            grants_with_payload.append({
+                'grant_id': grant_id,
+                'grant_slug': grant.slug,
+                'payload': payload
+            })
         except Exception as e:
             failures.append({
                 'active': 'grant_error',
@@ -2153,13 +2157,13 @@ def bulk_fund(request):
 
         successes.append({
             'title': _('Fund - Grant Funding Processed Successfully'),
-            'grant':grant_id,
+            'grant': grant_id,
             'text': _('Funding for this grant was successfully processed and saved.'),
             'success': True
         })
-        batch_grants_mail.append(grant_id)
 
-    # thank_you_for_supporting(batch_grants_mail, profile)
+    from grants.tasks import batch_process_grant_contributions
+    batch_process_grant_contributions.delay(grants_with_payload, profile.pk)
 
     return JsonResponse({
         'success': True,

app/marketing/mails.py

@@ -301,20 +301,21 @@ def new_supporter(grant, subscription):
         translation.activate(cur_language)
 
 
-def thank_you_for_supporting(grant, subscription):
-    if subscription and subscription.negative:
+def thank_you_for_supporting(grants_with_subscription):
+    positive_subscriptions = list(filter(lambda gws: not gws["subscription"].negative, grants_with_subscription))
+    if len(positive_subscriptions) == 0:
         return
 
     from_email = settings.CONTACT_EMAIL
-    to_email = subscription.contributor_profile.email
+    to_email = positive_subscriptions[0]["subscription"].contributor_profile.email
     if not to_email:
-        to_email = subscription.contributor_profile.user.email
+        to_email = positive_subscriptions[0]["subscription"].contributor_profile.user.email
 
     cur_language = translation.get_language()
 
     try:
         setup_lang(to_email)
-        html, text, subject = render_thank_you_for_supporting_email(grant, subscription)
+        html, text, subject = render_thank_you_for_supporting_email(grants_with_subscription)
 
         if not should_suppress_notification_email(to_email, 'thank_you_for_supporting'):
             send_mail(from_email, to_email, subject, text, html, categories=['transactional', func_name()])

app/retail/emails.py

@@ -131,8 +131,8 @@ def render_new_supporter_email(grant, subscription):
     return response_html, response_txt, subject
 
 
-def render_thank_you_for_supporting_email(grant, subscription):
-    params = {'grant': grant, 'subscription': subscription}
+def render_thank_you_for_supporting_email(grants_with_subscription):
+    params = {'grants_with_subscription': grants_with_subscription}
     response_html = premailer_transform(render_to_string("emails/grants/thank_you_for_supporting.html", params))
     response_txt = render_to_string("emails/grants/thank_you_for_supporting.txt", params)
     subject = _("Thank you for supporting Grants on Gitcoin!")

app/retail/templates/emails/grants/thank_you_for_supporting.html

@@ -101,6 +101,8 @@
   }
 
 </style>
+{% for grant_with_subscription in grants_with_subscription %}
+{% with subscription=grant_with_subscription.subscription grant=grant_with_subscription.grant %}
 <div class="grant-header">
   <div class="left-grant-header">
     <img src="{% static "v2/images/heart-robot.png" %}" alt="{% trans "Gitcoin Heart Robot" %}" title="{% trans "Gitcoin Heart Robot" %}">
@@ -145,4 +147,6 @@ <h2 style="text-transform: none;">{% trans "The world of open source is a better
   <a href="{% url 'grants:subscription_cancel' grant.id grant.slug subscription.id %}">{% trans "here." %}</a>
 </p>
 <hr>
+{% endwith %}
+{% endfor %}
 {% endblock %}

app/retail/templates/emails/grants/thank_you_for_supporting.txt

@@ -2,7 +2,8 @@
 
 {% trans "Gitcoin Heart Robot" %}
 
-{% for subscription in subscriptions %}
+{% for grant_with_subscription in grants_with_subscription %}
+{% with subscription=grant_with_subscription.subscription grant=grant_with_subscription.grant %}
 {% trans "Thank you for supporting" %} {{ subscription.grant.title }}
 
 {% trans "The world of open source is a better place because of you." %}
@@ -25,4 +26,5 @@
 {% trans "View Updates" %}
 
 {% trans "If you ever need to you can cancel your support" %} {% trans "here." %}
+{% endwith %}
 {% endfor %}