From e6787ff665102440db2be3907335b38487d36b6a Mon Sep 17 00:00:00 2001 From: Craig Oates Date: Wed, 10 Feb 2021 19:02:29 +0000 Subject: [PATCH] add extra return statement for gallery device status. In UpdateStatusColour, the code checks to see what state the device it's processing is in and relays that information to the user with a colour change on the dashboard. In a previous commit, I forgot to include a return statement to indicate the Relays (gallery1 and 2) are on. The one line of code added here addresses that. Note: The code is starting to look sloppy. The code in UpdateStatusColour is beginning to turn into a sprawing mess of branching logic. I have decided to keep like this for now because of the fixed requirements but you should be aware of the mess building-up. If the requirements begin to change, here might be a good place to start refactoring from. --- src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest | 2 +- src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest b/src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest index a0932cf..80bcd39 100644 --- a/src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest +++ b/src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest @@ -9,7 +9,7 @@ + Version="2021.2.10.0" /> diff --git a/src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs b/src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs index b1dec6e..619f2ba 100644 --- a/src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs +++ b/src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs @@ -112,6 +112,8 @@ namespace EyesAndEars.UWP.Services { // The device is on but factory lights are off. return new SolidColorBrush(Colors.LightSkyBlue); } + // Indicate the gallery relays are on. + return new SolidColorBrush(Colors.LightSeaGreen); } // The device is off. return new SolidColorBrush(Colors.DarkRed);