search

jontofront / ecoNET-300-Home-Assistant-Integration

ecoNET 300 integration to HA
11 stars 4 forks source link

Feature rewrite endpoint and logic extended #40

Closed AkiraJager closed 11 months ago

AkiraJager commented 11 months ago

This code is not finished, binary sensors need reMake based on new logic

Aditionaly cleanup is needed, will perform it sometime later

jontofront commented 11 months ago

Is it for testing?

AkiraJager commented 11 months ago

Is it for testing?

You can test it out to sees if all of described sensors are apearing corectly. If no issue appear, i would crate and start a PR with binary sensors

jontofront commented 11 months ago

image

AkiraJager commented 11 months ago

image

Okay i now see the issue here, problem is that when we are trying to add Entities to HA we get an error becauses of

Coordinator.data has keys like [1,2,3,4,5] becauses they are returned from Api and our Entities have keys like 'heatPump' because i tried to convert them for readabilyti

WIll leave the original intereger key and use translations

@jontofront Will make the patch in upcoming days