Merge pull request #20046 from mozilla/fix-subscription-reminders-args

fix(subscription-reminders): Fix misconfigured subscription reminder args

Author: david1alvarez

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

@@ -40,6 +40,7 @@ interface EndingRemindersOptions {
 }
 
 interface RenewalRemindersOptions {
+  monthlyReminderDays?: number;
   yearlyReminderDays?: number;
 }
 
@@ -81,6 +82,11 @@ export class SubscriptionReminders {
     this.statsd = statsd;
     this.planDuration = Duration.fromObject({ days: planLength });
     this.reminderDuration = Duration.fromObject({ days: reminderLength });
+    if (renewalRemindersOptions.monthlyReminderDays) {
+      this.reminderDuration = Duration.fromObject({
+        days: renewalRemindersOptions.monthlyReminderDays,
+      });
+    }
     if (renewalRemindersOptions.yearlyReminderDays) {
       this.yearlyRenewalReminderDuration = Duration.fromObject({
         days: renewalRemindersOptions.yearlyReminderDays,

packages/fxa-auth-server/scripts/subscription-reminders.ts

@@ -87,7 +87,7 @@ async function init() {
   const subscriptionReminders = new SubscriptionReminders(
     log,
     parseInt(program.planLength),
-    parseInt(program.reminderLength),
+    parseInt(program.monthlyRenewalReminderLength),
     {
       enabled: parseBooleanArg(program.enableEndingReminders),
       paymentsNextUrl: config.smtp.subscriptionSettingsUrl,
@@ -96,6 +96,7 @@ async function init() {
       yearlyReminderDays: parseInt(program.endingReminderYearlyLength),
     },
     {
+      monthlyReminderDays: parseInt(program.monthlyRenewalReminderLength),
       yearlyReminderDays: parseInt(program.yearlyRenewalReminderLength),
     },
     database,