Skip to content

Commit

Permalink
#1379 dont duplicate/fire newSubscriberEmailNotification twice
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisjsimpson committed Jul 21, 2024
1 parent e02fd89 commit aa755cf
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion subscribie/email.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def send_welcome_email(to_email=None, subscription=None, **kwargs):
plan=plan,
subscriber_email=to_email,
)

log.info(f"send_welcome_email rendered as:\n{html}")
send_email(
to_email=to_email,
subject=company.name + " " + "Subscription Confirmation",
Expand Down
1 change: 0 additions & 1 deletion subscribie/signals.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ def register_signal_handlers():
"""Connect signals to recievers
This is called during flask app startup in views.py
"""
signal_journey_complete.connect(newSubscriberEmailNotification)
signal_new_subscriber.connect(receiver_new_subscriber)
signal_journey_complete.connect(receiver_attach_documents_to_subscription)
signal_payment_failed.connect(
Expand Down

0 comments on commit aa755cf

Please sign in to comment.