Process "FWE" and use "mdi:electric-switch" for relay state

hoylabs · Nov 18, 2024 · 48c00d9 · 48c00d9
1 parent dfb9edd
commit 48c00d9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp b/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp
@@ -253,6 +253,13 @@ void VeDirectFrameHandler<T>::processTextData(std::string const& name, std::stri
 		return;
 	}
 
+    // According "VE.Direct-Protocol-3.33.pdf" for "Phoenix Charger" and "Smart BuckBoost".
+    // But it seems also some MPPT deliver FWE.
+	if (name == "FWE") {
+        _msgOut->printf("%s FWE ignored. We use FE to process the firmware version (value '%s')\r\n", _logId, value.c_str());
+		return;
+	}
+
 	if (name == "V") {
 		_tmpFrame.batteryVoltage_V_mV = atol(value.c_str());
 		return;

diff --git a/src/MqttHandleVedirectHass.cpp b/src/MqttHandleVedirectHass.cpp
@@ -89,7 +89,7 @@ void MqttHandleVedirectHassClass::publishConfig()
 
         // optional info, provided only if the charge controller delivers the information
         if (optMpptData->relayState_RELAY.first != 0) {
-            publishBinarySensor("MPPT error relay state", "mdi:export", "RELAY", "ON", "OFF", *optMpptData);
+            publishBinarySensor("MPPT error relay state", "mdi:electric-switch", "RELAY", "ON", "OFF", *optMpptData);
         }
         if (optMpptData->loadOutputState_LOAD.first != 0) {
             publishBinarySensor("MPPT load output state", "mdi:export", "LOAD", "ON", "OFF", *optMpptData);