nstrelow / ha_philips_android_tv

Home Assistant custom component for the newer (2016+) Philips Android TVs
109 stars 28 forks source link

Wake on Lan improvements #21

Closed Wykks closed 5 years ago

Wykks commented 5 years ago

Move wakeonlan import (as specified here https://developers.home-assistant.io/docs/en/0.85/creating_component_deps_and_reqs.html#note). It doesn't matters much, because wakeonlan is already loaded by home assistant. That's why it does work even if the import was not in the right place.

Split turn on function in two step & add more meaningful log infos/warn/error. (One of my issue was that I was not running the docker container in host network).

I have been running this for about two weeks now. The "turn on" command works perfectly. Using OLED903 & LAN WOL (wifi disabled).

Of course, ideally we want to make this async, but if I understand correctly, the whole component should be rewritten to async.