From fdb892ff6bbcacecac55c0a95eabfff7cb630e6b Mon Sep 17 00:00:00 2001 From: bl00dymarie Date: Tue, 23 Jan 2024 16:07:21 +0100 Subject: [PATCH] Disable tempReminder if temp is turned off --- components/settings/reminders/reminders.js | 17 ++++++++++++++--- .../settings/reminders/temperature-reminder.js | 2 ++ i18n/en/settings.js | 5 +++++ 3 files changed, 21 insertions(+), 3 deletions(-) diff --git a/components/settings/reminders/reminders.js b/components/settings/reminders/reminders.js index 518886b..1279d29 100644 --- a/components/settings/reminders/reminders.js +++ b/components/settings/reminders/reminders.js @@ -9,6 +9,7 @@ import { periodReminderObservable, savePeriodReminder, periodPredictionObservable, + temperatureTrackingCategoryObservable, } from '../../../local-storage' import labels from '../../../i18n/en/settings' @@ -34,6 +35,14 @@ const Reminders = () => { } } + const tempReminderDisabledPrompt = () => { + if (!temperatureTrackingCategoryObservable.value) { + Alert.alert( + labels.tempReminder.alertNoTempReminder.title, + labels.tempReminder.alertNoTempReminder.message + ) + } + } return ( @@ -46,9 +55,11 @@ const Reminders = () => { /> - - - + + + + + ) } diff --git a/components/settings/reminders/temperature-reminder.js b/components/settings/reminders/temperature-reminder.js index d5fb874..5b06b19 100644 --- a/components/settings/reminders/temperature-reminder.js +++ b/components/settings/reminders/temperature-reminder.js @@ -7,6 +7,7 @@ import AppSwitch from '../../common/app-switch' import { saveTempReminder, tempReminderObservable, + temperatureTrackingCategoryObservable, } from '../../../local-storage' import padWithZeros from '../../helpers/pad-time-with-zeros' @@ -51,6 +52,7 @@ const TemperatureReminder = () => { onToggle={temperatureReminderToggle} text={tempReminderText} value={isEnabled} + disabled={!temperatureTrackingCategoryObservable.value} /> `Daily reminder set for ${time}`, notification: 'Record your morning temperature', + alertNoTempReminder: { + title: 'Temperature turned off', + message: + 'To use the temperature reminder please first enable the temperature tracking category in the customization settings.', + }, }, periodReminder: { title: 'Next period reminder',