fix: Fix Email notifications to multiple email addresses (#1437)

Previously, sending notifications to multiple email addresses (not users) at once didn't work.
pinterest · Apr 12, 2024 · 2dbc30d · 2dbc30d
1 parent 4c74ac8
commit 2dbc30d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/querybook/server/lib/notify/notifier/email_notifier.py b/querybook/server/lib/notify/notifier/email_notifier.py
@@ -33,11 +33,11 @@ def notify_recipients(self, recipients, message):
             msg["Subject"] = subject
             msg["Date"] = date
             msg["From"] = from_email
-            msg["To"] = ",".join(recipients)
+            msg["To"] = ", ".join(recipients)
             msg.attach(MIMEText(message, "html"))
 
             smtp = smtplib.SMTP(QuerybookSettings.EMAILER_CONN)
-            smtp.sendmail(msg["From"], msg["To"], msg.as_string())
+            smtp.sendmail(msg["From"], recipients, msg.as_string())
         except Exception as e:
             LOG.info(e)