Skip to content

Commit a4e8363

Browse files
committed
fix(auth): free trial duplicate reminder emails
Because: - Free trial reminder emails are sending duplicate emails. This commit: - Update call of alreadySentEmail, by free trial logic, to pass in required current_period_start instead of current_period_end. Closes #PAY-3652
1 parent 0a78671 commit a4e8363

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

packages/fxa-auth-server/lib/payments/subscription-reminders.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,8 @@ export class SubscriptionReminders {
119119
days: endingReminderOptions.freeTrialReminderDays,
120120
});
121121
}
122-
this.freeTrialEndingReminderEnabled = endingReminderOptions.freeTrialEndRemindersEnabled;
122+
this.freeTrialEndingReminderEnabled =
123+
endingReminderOptions.freeTrialEndRemindersEnabled;
123124
this.paymentsNextUrl = endingReminderOptions.paymentsNextUrl;
124125
this.stripeHelper = stripeHelper;
125126
this.subscriptionManager = subscriptionManager;
@@ -313,7 +314,7 @@ export class SubscriptionReminders {
313314
if (
314315
await this.alreadySentEmail(
315316
uid,
316-
Math.floor(subscription.current_period_end * 1000),
317+
Math.floor(subscription.current_period_start * 1000),
317318
emailParams,
318319
'freeTrialEndingReminder'
319320
)

0 commit comments

Comments
 (0)