chore: update org cancelled alert to only send to customersuccess@fla…

…gsmith.com (#4522)

api/app/settings/common.py

@@ -1238,3 +1238,7 @@
     "EDGE_V2_MIGRATION_READ_CAPACITY_BUDGET",
     default=0,
 )
+
+ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST = env.list(
+    "ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST", default=[]
+)

api/organisations/tasks.py

@@ -4,6 +4,7 @@
 
 from app_analytics.influxdb_wrapper import get_current_api_usage
 from django.conf import settings
+from django.core.mail import send_mail
 from django.db.models import F, Max, Q
 from django.utils import timezone
 from task_processor.decorators import (
@@ -73,10 +74,15 @@ def send_org_subscription_cancelled_alert(
     organisation_name: str,
     formatted_cancellation_date: str,
 ) -> None:
-    FFAdminUser.send_alert_to_admin_users(
-        subject=f"Organisation {organisation_name} has cancelled their subscription",
-        message=f"Organisation {organisation_name} has cancelled their subscription on {formatted_cancellation_date}",
-    )
+    if recipient_list := settings.ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST:
+        send_mail(
+            subject=f"Organisation {organisation_name} has cancelled their subscription",
+            message=f"Organisation {organisation_name} has cancelled their subscription "
+            f"on {formatted_cancellation_date}",
+            from_email=settings.DEFAULT_FROM_EMAIL,
+            recipient_list=recipient_list,
+            fail_silently=True,
+        )
 
 
 @register_recurring_task(

api/tests/unit/organisations/test_unit_organisations_tasks.py

@@ -253,9 +253,14 @@ def test_finish_subscription_cancellation(db: None, mocker: MockerFixture) -> No
     assert organisation4.num_seats == organisation_user_count
 
 
-def test_send_org_subscription_cancelled_alert(db: None, mocker: MockerFixture) -> None:
+def test_send_org_subscription_cancelled_alert(
+    mocker: MockerFixture, settings: SettingsWrapper
+) -> None:
     # Given
-    send_mail_mock = mocker.patch("users.models.send_mail")
+    send_mail_mock = mocker.patch("organisations.tasks.send_mail")
+
+    recipient = "[email protected]"
+    settings.ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST = [recipient]
 
     # When
     send_org_subscription_cancelled_alert(
@@ -268,7 +273,7 @@ def test_send_org_subscription_cancelled_alert(db: None, mocker: MockerFixture)
         subject="Organisation TestCorp has cancelled their subscription",
         message="Organisation TestCorp has cancelled their subscription on 2023-02-08 09:12:34",
         from_email="[email protected]",
-        recipient_list=[],
+        recipient_list=[recipient],
         fail_silently=True,
     )
 

api/tests/unit/organisations/test_unit_organisations_views.py

@@ -622,6 +622,7 @@ def test_when_subscription_is_set_to_non_renewing_then_cancellation_date_set_and
     subscription: Subscription,
     staff_user: FFAdminUser,
     staff_client: APIClient,
+    settings: SettingsWrapper,
 ) -> None:
     # Given
     cancellation_date = datetime.now(tz=UTC) + timedelta(days=1)
@@ -640,6 +641,8 @@ def test_when_subscription_is_set_to_non_renewing_then_cancellation_date_set_and
     }
     url = reverse("api-v1:chargebee-webhook")
 
+    settings.ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST = ["[email protected]"]
+
     # When
     staff_client.post(url, data=json.dumps(data), content_type="application/json")
 
@@ -658,6 +661,7 @@ def test_when_subscription_is_cancelled_then_cancellation_date_set_and_alert_sen
     subscription: Subscription,
     staff_user: FFAdminUser,
     staff_client: APIClient,
+    settings: SettingsWrapper,
 ) -> None:
     # Given
     cancellation_date = datetime.now(tz=UTC) + timedelta(days=1)
@@ -676,6 +680,8 @@ def test_when_subscription_is_cancelled_then_cancellation_date_set_and_alert_sen
     }
     url = reverse("api-v1:chargebee-webhook")
 
+    settings.ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST = ["[email protected]"]
+
     # When
     staff_client.post(url, data=json.dumps(data), content_type="application/json")
 

infrastructure/aws/production/ecs-task-definition-task-processor.json

@@ -168,6 +168,10 @@
                 {
                     "name": "ENABLE_API_USAGE_ALERTING",
                     "value": "True"
+                },
+                {
+                    "name": "ORG_SUBSCRIPTION_CANCELLED_ALERT_RECIPIENT_LIST",
+                    "value": "[email protected]"
                 }
             ],
             "secrets": [