1
0
Fork 0
Browse Source

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.
stable
Craig Oates 3 years ago
parent
commit
e6787ff665
  1. 2
      src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest
  2. 2
      src/EyesAndEars.UWP/EyesAndEars.UWP/Services/DataServices.cs

2
src/EyesAndEars.UWP/EyesAndEars.UWP/Package.appxmanifest

@ -9,7 +9,7 @@
<Identity
Name="55acd946-60d4-4776-b6c6-03fef750e3da"
Publisher="CN=Craig Oates"
Version="2021.2.7.0" />
Version="2021.2.10.0" />
<mp:PhoneIdentity PhoneProductId="55acd946-60d4-4776-b6c6-03fef750e3da" PhonePublisherId="00000000-0000-0000-0000-000000000000"/>

2
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);