turbokongen / hass-AMS

Custom component reading AMS through MBus adapter into HomeAssistant
44 stars 10 forks source link

Bring back YAML configuration option #20

Closed turbokongen closed 4 years ago

Hellowlol commented 4 years ago

IMO setting up using yaml should never been removed. Im still pretty sure the error is in _check and update method. I can check and make a PR for this.

turbokongen commented 4 years ago

IMO setting up using yaml should never been removed. Im still pretty sure the error is in _check and update method. I can check and make a PR for this.

Sure, no problem. Let's add it back in.

turbokongen commented 4 years ago

@Hellowlol Can you test this? I've tested it on my machines, and works as expected.

Hellowlol commented 4 years ago

Will do. Btw i think you should change the name in hacs.json i couldnt figure out why i didnt find the integration when i tried to set it up.

turbokongen commented 4 years ago

@Hellowlol did this work to your expectations?

Hellowlol commented 4 years ago

I havn't got around to test it. Busy with some other HA stuff. Ill test it later to day.