fix: add param info, clean up trailing whitespace and minor formatting

src/PepperDash.Core/Device.cs

@@ -15,23 +15,23 @@ namespace PepperDash.Core
 		/// Unique Key
 		/// </summary>
 		public string Key { get; protected set; }
-  /// <summary>
-  /// Gets or sets the Name
-  /// </summary>
+		/// <summary>
+		/// Gets or sets the Name
+		/// </summary>
 		public string Name { get; protected set; }
 		/// <summary>
-		/// 
+		/// Gets or sets a value indicating whether the device is enabled
 		/// </summary>
 		public bool Enabled { get; protected set; }
 
-		/// <summary>
-		/// A place to store reference to the original config object, if any. These values should 
-		/// NOT be used as properties on the device as they are all publicly-settable values.
-		/// </summary>
+		// /// <summary>
+		// /// A place to store reference to the original config object, if any. These values should 
+		// /// NOT be used as properties on the device as they are all publicly-settable values.
+		// /// </summary>
 		//public DeviceConfig Config { get; private set; }
-		/// <summary>
-		/// Helper method to check if Config exists
-		/// </summary>
+		// /// <summary>
+		// /// Helper method to check if Config exists
+		// /// </summary>
 		//public bool HasConfig { get { return Config != null; } }
 
 		List<Action> _PreActivationActions;

src/PepperDash.Core/Logging/DebugConsoleSink.cs

@@ -41,6 +41,9 @@ namespace PepperDash.Core
             CrestronConsole.PrintLine(message);
         }
 
+        /// <summary>
+        /// Constructor for DebugConsoleSink        
+        /// </summary>
         public DebugConsoleSink(ITextFormatter formatProvider )
         {
             _textFormatter = formatProvider ?? new JsonFormatter();
@@ -48,6 +51,9 @@ namespace PepperDash.Core
 
     }
 
+    /// <summary>
+    /// Provides extension methods for DebugConsoleSink
+    /// </summary>
     public static class DebugConsoleSinkExtensions
     {
         /// <summary>

src/PepperDash.Core/Logging/DebugCrestronLoggerSink.cs

@@ -27,6 +27,9 @@ namespace PepperDash.Core.Logging
             CrestronLogger.WriteToLog(message, (uint)logEvent.Level);
         }
 
+        /// <summary>
+        /// Constructor for DebugCrestronLoggerSink
+        /// </summary>
         public DebugCrestronLoggerSink()
         {
             CrestronLogger.Initialize(1, LoggerModeEnum.RM);

src/PepperDash.Core/Logging/DebugErrorLogSink.cs

@@ -63,6 +63,10 @@ namespace PepperDash.Core.Logging
             handler(message);
         }
 
+        /// <summary>
+        /// Constructor for DebugErrorLogSink
+        /// </summary>
+        /// <param name="formatter">text formatter for log output</param>
         public DebugErrorLogSink(ITextFormatter formatter = null)
         {
             _formatter = formatter;

src/PepperDash.Core/Logging/DebugExtensions.cs

@@ -4,6 +4,9 @@ using Log = PepperDash.Core.Debug;
 
 namespace PepperDash.Core.Logging
 {
+    /// <summary>
+    /// Provides extension methods for logging on IKeyed objects
+    /// </summary>
     public static class DebugExtensions
     {
         /// <summary>

src/PepperDash.Core/Logging/DebugWebsocketSink.cs

@@ -32,6 +32,9 @@ namespace PepperDash.Core
         private const string _certificateName = "selfCres";
         private const string _certificatePassword = "cres12345";
 
+        /// <summary>
+        /// Gets the Port
+        /// </summary>
         public int Port 
         { get 
             { 
@@ -41,6 +44,9 @@ namespace PepperDash.Core
             } 
         }
 
+        /// <summary>
+        /// Gets the Url
+        /// </summary>
         public string Url
         {
             get
@@ -58,6 +64,11 @@ namespace PepperDash.Core
 
         private readonly ITextFormatter _textFormatter;
 
+        /// <summary>
+        /// Constructor for DebugWebsocketSink
+        /// </summary>
+        /// <param name="formatProvider">text formatter for log output</param>
+      
         public DebugWebsocketSink(ITextFormatter formatProvider)
         {
 
@@ -217,6 +228,9 @@ namespace PepperDash.Core
         }
     }
 
+    /// <summary>
+    /// Provides extension methods for DebugWebsocketSink
+    /// </summary>
     public static class DebugWebsocketSinkExtensions
     {
         /// <summary>
@@ -237,6 +251,9 @@ namespace PepperDash.Core
     {
         private DateTime _connectionTime;
 
+        /// <summary>
+        /// Gets the ConnectedDuration
+        /// </summary>
         public TimeSpan ConnectedDuration
         {
             get
@@ -252,11 +269,17 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// Constructor for DebugClient
+        /// </summary>
         public DebugClient()
         {
             Debug.Console(0, "DebugClient Created");
         }
 
+        /// <summary>
+        /// OnOpen method
+        /// </summary>
         protected override void OnOpen()
         {
             base.OnOpen();
@@ -267,6 +290,9 @@ namespace PepperDash.Core
             _connectionTime = DateTime.Now;
         }
 
+        /// <summary>
+        /// OnMessage method
+        /// </summary>
         protected override void OnMessage(MessageEventArgs e)
         {
             base.OnMessage(e);
@@ -274,6 +300,9 @@ namespace PepperDash.Core
             Debug.Console(0, "WebSocket UiClient Message: {0}", e.Data);
         }
 
+        /// <summary>
+        /// OnClose method
+        /// </summary>
         protected override void OnClose(CloseEventArgs e)
         {
             base.OnClose(e);
@@ -282,6 +311,9 @@ namespace PepperDash.Core
 
         }
 
+        /// <summary>
+        /// OnError method
+        /// </summary>
         protected override void OnError(WebSocketSharp.ErrorEventArgs e)
         {
             base.OnError(e);

src/PepperDash.Core/Web/RequestHandlers/WebApiBaseRequestAsyncHandler.cs

@@ -5,9 +5,16 @@ using System.Threading.Tasks;
 
 namespace PepperDash.Core.Web.RequestHandlers
 {
+    /// <summary>
+    /// CWS Base Async Handler, implements IHttpCwsHandler
+    /// </summary>
     public abstract class WebApiBaseRequestAsyncHandler:IHttpCwsHandler
     {
         private readonly Dictionary<string, Func<HttpCwsContext, Task>> _handlers;
+       
+        /// <summary>
+        /// Indicates whether CORS is enabled
+        /// </summary>
         protected readonly bool EnableCors;
 
         /// <summary>

src/PepperDash.Core/Web/RequestHandlers/WebApiBaseRequestHandler.cs

@@ -10,6 +10,10 @@ namespace PepperDash.Core.Web.RequestHandlers
 	public abstract class WebApiBaseRequestHandler : IHttpCwsHandler
 	{
 		private readonly Dictionary<string, Action<HttpCwsContext>> _handlers;
+		
+		/// <summary>
+		/// Indicates whether CORS is enabled
+		/// </summary>
 		protected readonly bool EnableCors;
 
 		/// <summary>

src/PepperDash.Core/Web/WebApiServer.cs

@@ -45,9 +45,9 @@ namespace PepperDash.Core.Web
   /// </summary>
 		public bool IsRegistered { get; private set; }
 
-		/// <summary>
-		/// Http request handler
-		/// </summary>
+		// /// <summary>
+		// /// Http request handler
+		// /// </summary>
 		//public IHttpCwsHandler HttpRequestHandler
 		//{
 		//    get { return _server.HttpRequestHandler; }
@@ -58,9 +58,9 @@ namespace PepperDash.Core.Web
 		//    }
 		//}
 
-		/// <summary>
-		/// Received request event handler
-		/// </summary>
+		// /// <summary>
+		// /// Received request event handler
+		// /// </summary>
 		//public event EventHandler<HttpCwsRequestEventArgs> ReceivedRequestEvent
 		//{
 		//    add { _server.ReceivedRequestEvent += new HttpCwsRequestEventHandler(value); }

src/PepperDash.Essentials.Core/CrestronIO/GenericDigitalInputDevice.cs

@@ -21,7 +21,6 @@ namespace PepperDash.Essentials.Core.CrestronIO
     public class GenericDigitalInputDevice : EssentialsBridgeableDevice, IDigitalInput, IHasFeedback
     {
         private DigitalInput inputPort;
-        private bool invertState;
 
         /// <summary>
         /// Gets or sets the InputStateFeedback
@@ -42,10 +41,7 @@ namespace PepperDash.Essentials.Core.CrestronIO
             IOPortConfig config)
             : base(key, name)
         {
-            invertState = !string.IsNullOrEmpty(config.CircuitType) && 
-                         config.CircuitType.Equals("NC", StringComparison.OrdinalIgnoreCase);
-            
-            InputStateFeedback = new BoolFeedback("inputState", () => invertState ? !inputPort.State : inputPort.State);
+            InputStateFeedback = new BoolFeedback("inputState", () => inputPort.State);
 
             AddPostActivationAction(() =>
             {

src/PepperDash.Essentials.Core/CrestronIO/GenericVersiportInputDevice.cs

@@ -18,7 +18,6 @@ namespace PepperDash.Essentials.Core.CrestronIO
     public class GenericVersiportDigitalInputDevice : EssentialsBridgeableDevice, IDigitalInput, IPartitionStateProvider, IHasFeedback
     {
         private Versiport inputPort;
-        private bool invertState;
 
         /// <summary>
         /// Gets or sets the InputStateFeedback
@@ -48,10 +47,7 @@ namespace PepperDash.Essentials.Core.CrestronIO
         public GenericVersiportDigitalInputDevice(string key, string name, Func<IOPortConfig, Versiport> postActivationFunc, IOPortConfig config) :
             base(key, name)
         {
-            invertState = !string.IsNullOrEmpty(config.CircuitType) && 
-                         config.CircuitType.Equals("NC", StringComparison.OrdinalIgnoreCase);
-            
-            InputStateFeedback = new BoolFeedback("inputState", () => invertState ? !inputPort.DigitalIn : inputPort.DigitalIn);
+            InputStateFeedback = new BoolFeedback("inputState", () => inputPort.DigitalIn);
             PartitionPresentFeedback = new BoolFeedback("partitionPresent", () => !inputPort.DigitalIn);
 
             AddPostActivationAction(() =>

src/PepperDash.Essentials.Core/CrestronIO/IOPortConfig.cs

@@ -37,12 +37,5 @@ namespace PepperDash.Essentials.Core.CrestronIO
         /// </summary>
         [JsonProperty("minimumChange")]
         public int MinimumChange { get; set; }
-
-        /// <summary>
-        /// Gets or sets the circuit type: "NO" (Normally Open) or "NC" (Normally Closed)
-        /// If set to "NC", the input state will be inverted. Defaults to "NO" if not specified.
-        /// </summary>
-        [JsonProperty("circuitType")]
-        public string CircuitType { get; set; } = "NO";
     }
 }

src/PepperDash.Essentials.Core/Monitoring/StatusMonitorCollection.cs

@@ -14,7 +14,7 @@ using PepperDash.Core;
 namespace PepperDash.Essentials.Core
 {
 	/// <summary>
-	///
+	/// 
 	/// </summary>
 	public class StatusMonitorCollection : IStatusMonitor
 	{
@@ -59,61 +59,51 @@ namespace PepperDash.Essentials.Core
 
 		void ProcessStatuses()
 		{
-			var InError = Monitors.Where(m => m.Status == MonitorStatus.InError).ToList();
-			var InWarning = Monitors.Where(m => m.Status == MonitorStatus.InWarning).ToList();
-			var IsOk = Monitors.Where(m => m.Status == MonitorStatus.IsOk).ToList();
+			var InError = Monitors.Where(m => m.Status == MonitorStatus.InError);
+			var InWarning = Monitors.Where(m => m.Status == MonitorStatus.InWarning);
+			var IsOk = Monitors.Where(m => m.Status == MonitorStatus.IsOk);
 
 
 			MonitorStatus initialStatus;
 			string prefix = "0:";
-			if (InError.Any())
+			if (InError.Count() > 0)
 			{
 				initialStatus = MonitorStatus.InError;
 				prefix = "3:";
 			}
-			else if (InWarning.Any())
+			else if (InWarning.Count() > 0)
 			{
 				initialStatus = MonitorStatus.InWarning;
 				prefix = "2:";
 			}
-			else if (IsOk.Any())
+			else if (IsOk.Count() > 0)
 				initialStatus = MonitorStatus.IsOk;
 			else
 				initialStatus = MonitorStatus.StatusUnknown;
 
 			// Build the error message string
-			if (InError.Any() || InWarning.Any())
-			{
-				var errorNames = InError
-					.Select(mon => mon.Parent is IKeyName keyName ? keyName.Name : mon.Parent.Key)
-					.ToList();
-				var warningNames = InWarning
-					.Select(mon => mon.Parent is IKeyName keyName ? keyName.Name : mon.Parent.Key)
-					.ToList();
-
-				var sb = new StringBuilder(prefix);
-
-				if (errorNames.Count > 0)
-				{
-					sb.Append($"{errorNames.Count} Error{(errorNames.Count > 1 ? "s" : "")}: ");
-					sb.Append(string.Join(", ", errorNames));
-				}
-				if (warningNames.Count > 0)
-				{
-					if (errorNames.Count > 0)
-						sb.Append("; ");
-
-					sb.Append($"{warningNames.Count} Warning{(warningNames.Count > 1 ? "s" : "")}: ");
-					sb.Append(string.Join(", ", warningNames));
-				}
-
-                sb.Append(" Offline");
-				Message = sb.ToString();
-			}
-			else
-			{
-				Message = "Room Ok.";
-			}
+            if (InError.Count() > 0 || InWarning.Count() > 0)
+            {
+                StringBuilder sb = new StringBuilder(prefix);
+                if (InError.Count() > 0)
+                {
+                    // Do string splits and joins 
+                    sb.Append(string.Format("{0} Errors:", InError.Count()));
+                    foreach (var mon in InError)
+                        sb.Append(string.Format("{0}, ", mon.Parent.Key));
+                }
+                if (InWarning.Count() > 0)
+                {
+                    sb.Append(string.Format("{0} Warnings:", InWarning.Count()));
+                    foreach (var mon in InWarning)
+                        sb.Append(string.Format("{0}, ", mon.Parent.Key));
+                }
+                Message = sb.ToString();
+            }
+            else
+            {
+                Message = "Room Ok.";
+            }
 
 			// Want to fire even if status doesn't change because the message may.
 			Status = initialStatus;

src/PepperDash.Essentials.Devices.Common/Displays/ScreenLiftController.cs

@@ -19,7 +19,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
     {
         None,
         Raise,
-        Lower,
+        Lower
     }
 
     /// <summary>
@@ -50,8 +50,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             get { return _isInUpPosition; }
             set
             {
-                if (value == _isInUpPosition)
-                    return;
+                if (value == _isInUpPosition) return;
                 _isInUpPosition = value;
                 IsInUpPosition.FireUpdate();
                 PositionChanged?.Invoke(this, new EventArgs());
@@ -88,11 +87,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         /// <summary>
         /// Constructor for ScreenLiftController
         /// </summary>
-        public ScreenLiftController(
-            string key,
-            string name,
-            ScreenLiftControllerConfigProperties config
-        )
+        public ScreenLiftController(string key, string name, ScreenLiftControllerConfigProperties config)
             : base(key, name)
         {
             Config = config;
@@ -110,60 +105,27 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
-                {
-                    RaiseRelayConfig = Config.Relays["raise"];
-                    LowerRelayConfig = Config.Relays["lower"];
-                    break;
-                }
+                    {
+                        RaiseRelayConfig = Config.Relays["raise"];
+                        LowerRelayConfig = Config.Relays["lower"];
+                        break;
+                    }
                 case eScreenLiftControlMode.latched:
-                {
-                    LatchedRelayConfig = Config.Relays["latched"];
-                    break;
-                }
+                    {
+                        LatchedRelayConfig = Config.Relays["latched"];
+                        break;
+                    }
             }
-
-            IsInUpPosition.OutputChange += (sender, args) =>
-            {
-                this.LogDebug(
-                    "ScreenLiftController '{name}' IsInUpPosition changed to {position}",
-                    Name,
-                    IsInUpPosition.BoolValue ? "Up" : "Down"
-                );
-
-                if (!Config.MuteOnScreenUp)
-                {
-                    return;
-                }
-
-                if (args.BoolValue)
-                {
-                    return;
-                }
-
-                if (DisplayDevice is IBasicVideoMuteWithFeedback videoMute)
-                {
-                    this.LogInformation("Unmuting video because screen is down");
-                    videoMute.VideoMuteOff();
-                }
-            };
-
-            IsInUpPosition.FireUpdate();
         }
 
         private void IsCoolingDownFeedback_OutputChange(object sender, FeedbackEventArgs e)
         {
-            if (
-                !DisplayDevice.IsCoolingDownFeedback.BoolValue
-                && Type == eScreenLiftControlType.lift
-            )
+            if (!DisplayDevice.IsCoolingDownFeedback.BoolValue && Type == eScreenLiftControlType.lift)
             {
                 Raise();
                 return;
             }
-            if (
-                DisplayDevice.IsCoolingDownFeedback.BoolValue
-                && Type == eScreenLiftControlType.screen
-            )
+            if (DisplayDevice.IsCoolingDownFeedback.BoolValue && Type == eScreenLiftControlType.screen)
             {
                 Raise();
                 return;
@@ -188,18 +150,18 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
-                {
-                    this.LogDebug("Getting relays for {mode}", Mode);
-                    RaiseRelay = GetSwitchedOutputFromDevice(RaiseRelayConfig.DeviceKey);
-                    LowerRelay = GetSwitchedOutputFromDevice(LowerRelayConfig.DeviceKey);
-                    break;
-                }
+                    {
+                        this.LogDebug("Getting relays for {mode}", Mode);
+                        RaiseRelay = GetSwitchedOutputFromDevice(RaiseRelayConfig.DeviceKey);
+                        LowerRelay = GetSwitchedOutputFromDevice(LowerRelayConfig.DeviceKey);
+                        break;
+                    }
                 case eScreenLiftControlMode.latched:
-                {
-                    this.LogDebug("Getting relays for {mode}", Mode);
-                    LatchedRelay = GetSwitchedOutputFromDevice(LatchedRelayConfig.DeviceKey);
-                    break;
-                }
+                    {
+                        this.LogDebug("Getting relays for {mode}", Mode);
+                        LatchedRelay = GetSwitchedOutputFromDevice(LatchedRelayConfig.DeviceKey);
+                        break;
+                    }
             }
 
             this.LogDebug("Getting display with key {displayKey}", DisplayDeviceKey);
@@ -210,8 +172,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
                 this.LogDebug("Subscribing to {displayKey} feedbacks", DisplayDeviceKey);
 
                 DisplayDevice.IsWarmingUpFeedback.OutputChange += IsWarmingUpFeedback_OutputChange;
-                DisplayDevice.IsCoolingDownFeedback.OutputChange +=
-                    IsCoolingDownFeedback_OutputChange;
+                DisplayDevice.IsCoolingDownFeedback.OutputChange += IsCoolingDownFeedback_OutputChange;
             }
 
             return base.CustomActivate();
@@ -222,17 +183,10 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         /// </summary>
         public void Raise()
         {
-            if (RaiseRelay == null && LatchedRelay == null)
-                return;
+            if (RaiseRelay == null && LatchedRelay == null) return;
 
             this.LogDebug("Raise called for {type}", Type);
 
-            if (Config.MuteOnScreenUp && DisplayDevice is IBasicVideoMuteWithFeedback videoMute)
-            {
-                this.LogInformation("Muting video because screen is going up");
-                videoMute.VideoMuteOn();
-            }
-
             // If device is moving, bank the command
             if (_isMoving)
             {
@@ -246,33 +200,33 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
-                {
-                    PulseOutput(RaiseRelay, RaiseRelayConfig.PulseTimeInMs);
+                    {
+                        PulseOutput(RaiseRelay, RaiseRelayConfig.PulseTimeInMs);
 
-                    // Set moving flag and start timer if movement time is configured
-                    if (RaiseRelayConfig.MoveTimeInMs > 0)
-                    {
-                        _isMoving = true;
-                        _currentMovement = RequestedState.Raise;
-                        if (_movementTimer.Enabled)
+                        // Set moving flag and start timer if movement time is configured
+                        if (RaiseRelayConfig.MoveTimeInMs > 0)
                         {
-                            _movementTimer.Stop();
+                            _isMoving = true;
+                            _currentMovement = RequestedState.Raise;
+                            if (_movementTimer.Enabled)
+                            {
+                                _movementTimer.Stop();
+                            }
+                            _movementTimer.Interval = RaiseRelayConfig.MoveTimeInMs;
+                            _movementTimer.Start();
                         }
-                        _movementTimer.Interval = RaiseRelayConfig.MoveTimeInMs;
-                        _movementTimer.Start();
+                        else
+                        {
+                            InUpPosition = true;
+                        }
+                        break;
                     }
-                    else
-                    {
-                        InUpPosition = true;
-                    }
-                    break;
-                }
                 case eScreenLiftControlMode.latched:
-                {
-                    LatchedRelay.Off();
-                    InUpPosition = true;
-                    break;
-                }
+                    {
+                        LatchedRelay.Off();
+                        InUpPosition = true;
+                        break;
+                    }
             }
         }
 
@@ -281,8 +235,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         /// </summary>
         public void Lower()
         {
-            if (LowerRelay == null && LatchedRelay == null)
-                return;
+            if (LowerRelay == null && LatchedRelay == null) return;
 
             this.LogDebug("Lower called for {type}", Type);
 
@@ -299,33 +252,33 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             switch (Mode)
             {
                 case eScreenLiftControlMode.momentary:
-                {
-                    PulseOutput(LowerRelay, LowerRelayConfig.PulseTimeInMs);
+                    {
+                        PulseOutput(LowerRelay, LowerRelayConfig.PulseTimeInMs);
 
-                    // Set moving flag and start timer if movement time is configured
-                    if (LowerRelayConfig.MoveTimeInMs > 0)
-                    {
-                        _isMoving = true;
-                        _currentMovement = RequestedState.Lower;
-                        if (_movementTimer.Enabled)
+                        // Set moving flag and start timer if movement time is configured
+                        if (LowerRelayConfig.MoveTimeInMs > 0)
                         {
-                            _movementTimer.Stop();
+                            _isMoving = true;
+                            _currentMovement = RequestedState.Lower;
+                            if (_movementTimer.Enabled)
+                            {
+                                _movementTimer.Stop();
+                            }
+                            _movementTimer.Interval = LowerRelayConfig.MoveTimeInMs;
+                            _movementTimer.Start();
                         }
-                        _movementTimer.Interval = LowerRelayConfig.MoveTimeInMs;
-                        _movementTimer.Start();
+                        else
+                        {
+                            InUpPosition = false;
+                        }
+                        break;
                     }
-                    else
-                    {
-                        InUpPosition = false;
-                    }
-                    break;
-                }
                 case eScreenLiftControlMode.latched:
-                {
-                    LatchedRelay.On();
-                    InUpPosition = false;
-                    break;
-                }
+                    {
+                        LatchedRelay.On();
+                        InUpPosition = false;
+                        break;
+                    }
             }
         }
 
@@ -386,13 +339,16 @@ namespace PepperDash.Essentials.Devices.Common.Shades
         {
             output.On();
 
-            var timer = new Timer(pulseTime) { AutoReset = false };
+            var timer = new Timer(pulseTime)
+            {
+                AutoReset = false
+            };
 
             timer.Elapsed += (sender, e) =>
-            {
-                output.Off();
-                timer.Dispose();
-            };
+                  {
+                      output.Off();
+                      timer.Dispose();
+                  };
             timer.Start();
         }
 
@@ -405,10 +361,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             }
             else
             {
-                this.LogWarning(
-                    "Error: Unable to get relay device with key '{relayKey}'",
-                    relayKey
-                );
+                this.LogWarning("Error: Unable to get relay device with key '{relayKey}'", relayKey);
                 return null;
             }
         }
@@ -422,13 +375,11 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             }
             else
             {
-                this.LogWarning(
-                    "Error: Unable to get display device with key '{displayKey}'",
-                    displayKey
-                );
+                this.LogWarning("Error: Unable to get display device with key '{displayKey}'", displayKey);
                 return null;
             }
         }
+
     }
 
     /// <summary>
@@ -436,7 +387,7 @@ namespace PepperDash.Essentials.Devices.Common.Shades
     /// </summary>
     public class ScreenLiftControllerFactory : EssentialsDeviceFactory<RelayControlledShade>
     {
-        /// <summary>
+        /// <summary> 
         /// Constructor for ScreenLiftControllerFactory
         /// </summary>
         public ScreenLiftControllerFactory()
@@ -453,4 +404,4 @@ namespace PepperDash.Essentials.Devices.Common.Shades
             return new ScreenLiftController(dc.Key, dc.Name, props);
         }
     }
-}
+}

src/PepperDash.Essentials.Devices.Common/Displays/ScreenLiftControllerConfigProperties.cs

@@ -5,41 +5,37 @@ using PepperDash.Essentials.Core.DeviceTypeInterfaces;
 
 namespace PepperDash.Essentials.Devices.Common.Shades
 {
+  /// <summary>
+  /// Represents a ScreenLiftControllerConfigProperties
+  /// </summary>
+  public class ScreenLiftControllerConfigProperties
+  {
+
     /// <summary>
-    /// Represents a ScreenLiftControllerConfigProperties
+    /// Gets or sets the DisplayDeviceKey
     /// </summary>
-    public class ScreenLiftControllerConfigProperties
-    {
-        /// <summary>
-        /// Gets or sets the DisplayDeviceKey
-        /// </summary>
-        [JsonProperty("displayDeviceKey")]
-        public string DisplayDeviceKey { get; set; }
+    [JsonProperty("displayDeviceKey")]
+    public string DisplayDeviceKey { get; set; }
 
-        /// <summary>
-        /// Gets or sets the Type
-        /// </summary>
-        [JsonProperty("type")]
-        [JsonConverter(typeof(StringEnumConverter))]
-        public eScreenLiftControlType Type { get; set; }
+    /// <summary>
+    /// Gets or sets the Type
+    /// </summary>
+    [JsonProperty("type")]
+    [JsonConverter(typeof(StringEnumConverter))]
+    public eScreenLiftControlType Type { get; set; }
 
-        /// <summary>
-        /// Gets or sets the Mode
-        /// </summary>
-        [JsonProperty("mode")]
-        [JsonConverter(typeof(StringEnumConverter))]
-        public eScreenLiftControlMode Mode { get; set; }
+    /// <summary>
+    /// Gets or sets the Mode
+    /// </summary>
+    [JsonProperty("mode")]
+    [JsonConverter(typeof(StringEnumConverter))]
+    public eScreenLiftControlMode Mode { get; set; }
 
-        /// <summary>
-        /// Gets or sets the Relays
-        /// </summary>
-        [JsonProperty("relays")]
-        public Dictionary<string, ScreenLiftRelaysConfig> Relays { get; set; }
+    /// <summary>
+    /// Gets or sets the Relays
+    /// </summary>
+    [JsonProperty("relays")]
+    public Dictionary<string, ScreenLiftRelaysConfig> Relays { get; set; }
 
-        /// <summary>
-        /// Mutes the display when the screen is in the up position
-        /// </summary>
-        [JsonProperty("muteOnScreenUp")]
-        public bool MuteOnScreenUp { get; set; }
-    }
-}
+  }
+}