Merge pull request #47 from qgis/send_email_progress

Send email to the organisation owners when changing pending status

Author: Xpirix

django_project/certification/api_views/update_status.py

@@ -5,7 +5,9 @@
 from ..models.certifying_organisation import CertifyingOrganisation
 from ..models.status import Status
 from ..views import (
-    send_rejection_email, send_approved_email,
+    send_rejection_email,
+    send_approved_email,
+    send_pending_email,
     CertifyingOrganisationUserTestMixin
 )
 
@@ -76,6 +78,13 @@ def post(self, request, slug):
                     self.request.user.username
                 )
 
+            if status_id and status_name and status_name.lower() == "pending":
+                send_pending_email(
+                    certifyingorganisation,
+                    change_reason,
+                    site
+                )
+
             certifyingorganisation._change_reason = (
                 change_reason
             )

django_project/certification/tests/api_views/test_update_status.py

@@ -106,6 +106,9 @@ def test_update_status_access(self):
         )
         response = self.client.post(self.api_url, data)
         self.assertTrue(response.status_code, 200)
+        self.assertIn(
+            'Your organisation application status has been updated.',
+            mail.outbox[0].subject)
 
     @override_settings(VALID_DOMAIN=['testserver', ])
     def test_update_status_no_login(self):

django_project/certification/views/certifying_organisation.py

@@ -1117,7 +1117,7 @@ def send_approved_email(
             'project_slug': certifying_organisation.project.slug,
         }
         send_mail(
-            u'Projecta - Your organisation is approved',
+            u'QGIS Certification - Your organisation is approved',
             u'Dear {owner_firstname} {owner_lastname},\n\n'
             u'Congratulations!\n'
             u'Your certifying organisation has been approved. The '
@@ -1138,6 +1138,45 @@ def send_approved_email(
         )
 
 
+def send_pending_email(
+        certifying_organisation: CertifyingOrganisation,
+        change_reason: str,
+        site: request
+    ):
+    for organisation_owner in \
+            certifying_organisation.organisation_owners.all():
+        data = {
+            'owner_firstname': organisation_owner.first_name,
+            'owner_lastname': organisation_owner.last_name,
+            'change_reason': change_reason,
+            'organisation_name': certifying_organisation.name,
+            'project_name': certifying_organisation.project.name,
+            'project_owner_firstname':
+                certifying_organisation.project.owner.first_name,
+            'project_owner_lastname':
+                certifying_organisation.project.owner.last_name,
+            'site': site,
+            'project_slug': certifying_organisation.project.slug,
+            'details_url': 'https://' + site + reverse(
+                'certifyingorganisation-detail',
+                kwargs={'slug': certifying_organisation.slug}
+            )
+        }
+        send_mail(
+            u'QGIS Certification - Your organisation application status has been updated.',
+            u'Dear {owner_firstname} {owner_lastname},\n\n'
+            u'{change_reason}\n'
+            u'For further information please visit: '
+            u'{details_url}\n\n'
+            u'Sincerely,\n'
+            u'{project_owner_firstname} {project_owner_lastname}'.format(
+                **data),
+            certifying_organisation.project.owner.email,
+            [organisation_owner.email],
+            fail_silently=True,
+        )
+
+
 class ApproveCertifyingOrganisationView(
     CertifyingOrganisationMixin,
     RedirectView):