alexander-vitishchenko / hc3-to-mqtt

Fibaro HC3 integration with Home Assistant & NodeRed
MIT License
31 stars 13 forks source link

Danfoss LC13 TRV Support #17

Closed MomosX closed 1 year ago

MomosX commented 2 years ago

I have attached the json files for Danfoss LC13 raditor heads.

Thank you Danfoss-LC13.zip !

alexander-vitishchenko commented 2 years ago

Hi @MomosX

Can you please prioritize what thermostats take the most priority for you, if choosing between Fibaro FGT-001, Danfoss LC13 TRV Support, and Danfoss RoomSensor Thermostat?

MomosX commented 1 year ago

same fix https://github.com/alexander-vitishchenko/hc3-to-mqtt/issues/31 but use "com.fibaro.thermostatDanfoss"

alexander-vitishchenko commented 1 year ago

Hi @MomosX - can you please check if the latest version of the QuickApp works for your thermostats?

MomosX commented 1 year ago

works only if i modify:

function Thermostat.isSupported(fibaroDevice)
    if (fibaroDevice.type == "com.fibaro.hvacSystem" or fibaroDevice.type == "com.fibaro.FGT001" or fibaroDevice.type == "com.fibaro.thermostatDanfoss") then 
        return true 
    else 
        return false
    end
end
alexander-vitishchenko commented 1 year ago

Hi @MomosX are you sure that you use the latest version of the QuickApp?

I've tested your device JSON locally and it was successfully identified as the thermostat.

Screenshot 2023-01-08 230128

Can you please share your logs/screenshots?

alexander-vitishchenko commented 1 year ago

Hi @MomosX can you please advise if you are using the latest QuickApp version, and share full logs?

alexander-vitishchenko commented 1 year ago

Closed as expired, and need more inputs to confirm the issue exist and then fix it