added compatibility with pus7304 models

nstrelow / ha_philips_android_tv

Home Assistant custom component for the newer (2016+) Philips Android TVs

109 stars 28 forks source link

added compatibility with pus7304 models #38

Closed mikicof closed 4 years ago

mikicof commented 4 years ago

Fixes #32

I've found a workaround for this models that have sometimes some connectivity issues.

In this case when a tv lacks connections it sets the status to unknown instead of off so any automation about this status can be set specifically when the status changes to on from off and discarding connection drops.

May be also helpful for other models where the network is not stable.

nstrelow commented 4 years ago

AWESOME 🥳 Thanks for adding some failsafe code. I have no way of testing this right now, but I trust you tested it a lot.

Please post your fixes/ changes to the forum: https://community.home-assistant.io/t/philips-android-tv-component/17749. I am monitoring the forum and hope people will say something should it not work. Thanks again!

mikicof commented 4 years ago

I’m sorry I’ve found out this morning an extreme case it has happened to me when only fails the first call so the next one resets the api_online value to True, for the rest it has worked for me fine all the week, I’ve uploaded this to #39

I have tested this fix this afternoon and it seems to work fine, only triggering the automation when changing:

I will comment my changes to the forum hoping that this helps someone too :)