Merge pull request #110 from Jalle19/defrosting

Add setting switch for "defrosting allowed"
Jalle19 · Feb 12, 2024 · c953c36 · c953c36
2 parents 040a5fb + 7fca05c
commit c953c36
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,7 @@
 * Add optional sensors for control panel temperature, supply fan speed and exhaust fan speed (https://github.com/Jalle19/eda-modbus-bridge/issues/96)
 * Remove references to "flags", use "modes" everywhere instead
 * Use HTTP 500 instead of HTTP 400 for generic errors
+* Add settings switch for defrosting allowed (https://github.com/Jalle19/eda-modbus-bridge/issues/109)
 
 ## 2.7.1
 

diff --git a/app/enervent.mjs b/app/enervent.mjs
@@ -35,6 +35,7 @@ export const AVAILABLE_SETTINGS = {
     'awayHeatingAllowed': 18,
     'longAwayCoolingAllowed': 21,
     'longAwayHeatingAllowed': 20,
+    'defrostingAllowed': 55,
 }
 
 export const ALARM_REGISTERS_START = 385

diff --git a/app/homeassistant.mjs b/app/homeassistant.mjs
@@ -293,6 +293,11 @@ export const configureMqttDiscovery = async (modbusClient, mqttClient) => {
             // Not supported by some units
             { 'enabled_by_default': automationType !== AUTOMATION_TYPE_LEGACY_EDA }
         ),
+        'defrostingAllowed': createSettingSwitchConfiguration(
+            configurationBase,
+            'defrostingAllowed',
+            'Defrosting allowed'
+        ),
     }
 
     // Binary sensors for alarms

diff --git a/app/modbus.mjs b/app/modbus.mjs
@@ -220,6 +220,13 @@ export const getSettings = async (modbusClient) => {
         }
     }
 
+    // Defrosting allowed
+    result = await mutex.runExclusive(async () => tryReadCoils(modbusClient, 55, 1))
+    settings = {
+        ...settings,
+        'defrostingAllowed': result.data[0],
+    }
+
     return settings
 }
 
@@ -263,6 +270,7 @@ export const setSetting = async (modbusClient, setting, value) => {
         case 'awayHeatingAllowed':
         case 'longAwayCoolingAllowed':
         case 'longAwayHeatingAllowed':
+        case 'defrostingAllowed':
             coil = true
             break
     }