Merge branch '215-period-reminder-bug' into 'master'

wrongly triggered period reminders (bug) Closes #215 See merge request bloodyhealth/drip!92
2018-09-28 13:35:18 +00:00
parent 68f3faaf7c 107688ac36
commit fb5a51621f
1 changed files with 11 additions and 8 deletions
@@ -64,14 +64,17 @@ function setupPeriodReminder() {
      .hours(6)
      .minutes(0)
      .seconds(0)
-    // period is likely to start in 3 to 3 + (length of prediction - 1) days
-    const daysToEndOfPrediction = bleedingPrediction[0].length + 2

-    Notification.localNotificationSchedule({
-      id: '2',
-      message: labels.periodReminder.notification(daysToEndOfPrediction),
-      date: reminderDate.toDate(),
-      vibrate: false
-    })
+    if (reminderDate.isAfter()) {
+    // period is likely to start in 3 to 3 + (length of prediction - 1) days
+      const daysToEndOfPrediction = bleedingPrediction[0].length + 2
+
+      Notification.localNotificationSchedule({
+        id: '2',
+        message: labels.periodReminder.notification(daysToEndOfPrediction),
+        date: reminderDate.toDate(),
+        vibrate: false
+      })
+    }
  }
 }