#727 Sender email se večkrat pošlje (#143)

* addes todos * send email to sender only once * updated js import
velis74 · Oct 13, 2023 · 852df6b · 852df6b
1 parent 30eab3e
commit 852df6b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/django_project_base/notifications/base/channels/integrations/aws_ses.py b/django_project_base/notifications/base/channels/integrations/aws_ses.py
@@ -89,9 +89,9 @@ def client_send(self, sender: str, recipient: Recipient, msg: dict, dlr_id: str)
             .client("ses")
             .send_email(
                 Destination={
-                    "ToAddresses": [sender],
+                    "ToAddresses": [recipient.email],
                     "CcAddresses": [],
-                    "BccAddresses": [recipient.email],
+                    "BccAddresses": [],
                 },
                 Message=msg,
                 Source=sender,

diff --git a/django_project_base/notifications/base/channels/mail_channel.py b/django_project_base/notifications/base/channels/mail_channel.py
@@ -1,3 +1,4 @@
+import uuid
 from typing import List
 
 from django.conf import settings
@@ -24,6 +25,14 @@ def send(self, notification: DjangoProjectBaseNotification, extra_data, **kwargs
                 list(map(int, notification.recipients.split(","))) if notification.recipients else []
             )
             return len(recipients)
+        message = self.provider.get_message(notification)
+        sender = self.sender(notification)
+        self.provider.client_send(
+            self.sender(notification),
+            Recipient(identifier=str(uuid.uuid4()), phone_number="", email=sender),
+            message,
+            str(uuid.uuid4()),
+        )
         return super().send(notification=notification, extra_data=extra_data)
 
     def get_recipients(self, notification: DjangoProjectBaseNotification, unique_identifier=""):