From 283389230be51e302218c04ca4b4e10c926312be Mon Sep 17 00:00:00 2001 From: Birk Jernstrom Date: Fri, 3 Jan 2025 14:26:39 +0100 Subject: [PATCH] server: Remove duplicate logic check before sending subscription confirmation email --- server/polar/subscription/service.py | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/server/polar/subscription/service.py b/server/polar/subscription/service.py index d4e87ed9ad..bfd831d324 100644 --- a/server/polar/subscription/service.py +++ b/server/polar/subscription/service.py @@ -971,17 +971,7 @@ async def _on_subscription_activated( session, subscription, WebhookEventType.subscription_active ) - # TODO: Copied this logic over. Why the extra check for customer - # confirmation vs. webhook/creator notification? Look into it, but avoid - # changing for now. - if ( - subscription.started_at is not None - and subscription.started_at.date() - == subscription.current_period_start.date() - and not subscription.ends_at - ): - await self.send_confirmation_email(session, subscription) - + await self.send_confirmation_email(session, subscription) await self._send_new_subscription_notification(session, subscription) async def _on_subscription_uncanceled(