From b6719365b2fc8b253b060f38df55b3749775359c Mon Sep 17 00:00:00 2001 From: Philip Abbey Date: Tue, 19 Nov 2024 21:56:32 +0000 Subject: [PATCH] Changes as per issue. --- source/Settings.mc | 9 ++++++--- source/WebhookManager.mc | 4 ++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/source/Settings.mc b/source/Settings.mc index 06191a6..0529fb4 100644 --- a/source/Settings.mc +++ b/source/Settings.mc @@ -14,7 +14,12 @@ // // Description: // -// Home Assistant settings. +// Home Assistant settings. +// +// WARNING! +// +// Careful putting ErrorView.show() calls in here. They need to be guarded so that +// they do not get called when only displaying the glance view. // //----------------------------------------------------------------------------------- @@ -93,8 +98,6 @@ class Settings { Background.deleteTemporalEvent(); Background.deleteActivityCompletedEvent(); } - } else { - ErrorView.show(WatchUi.loadResource($.Rez.Strings.NoPhone) as Lang.String); } } else { // Explicitly disable the background event which persists when the application closes. diff --git a/source/WebhookManager.mc b/source/WebhookManager.mc index 8c3b31c..b25380b 100644 --- a/source/WebhookManager.mc +++ b/source/WebhookManager.mc @@ -260,7 +260,7 @@ class WebhookManager { ]; if (Toybox has :ActivityMonitor) { - System.println("WebhookManager registerWebhookSensors(): has ActivityMonitor class"); + // System.println("WebhookManager registerWebhookSensors(): has ActivityMonitor class"); var activityInfo = ActivityMonitor.getInfo(); sensors.add({ "name" => "Steps today", @@ -309,7 +309,7 @@ class WebhookManager { }); } } else { - System.println("WebhookManager registerWebhookSensors(): has no ActivityMonitor class"); + //System.println("WebhookManager registerWebhookSensors(): has no ActivityMonitor class"); } if (Activity has :getProfileInfo) {