search

Mmodarre / pyfujitsu_for_homeassistant

MIT License
31 stars 33 forks source link

Support for Home Assistant > 0.96 #13

Open xerxes87 opened 5 years ago

xerxes87 commented 5 years ago

Hi Mmodarre,

here the PR for the Home Assistant > 0.96 support, with fixed requirement.

Thanks for review!

BR philwilldo

meichthys commented 4 years ago

@xerxes87 Is this merge still valid? I tried your branch and i still seemed to run into the same error when validating the yaml:

image

chiefcomm commented 4 years ago

@Mmodarre the original developer hasn't been making updates since this integration broke back at version 0.95 - I suggest reading https://community.home-assistant.io/t/add-support-for-fujitsu-wireless-air-conditioning-control-app-fglair/65999/101 which explains the changes made by @xerxes87

meichthys commented 4 years ago

@Mmodarre hasn't made commits recently but he has had activity on issues as recent as July and has made contributions to the underlying pyfujitsu project as recent as November.

pilot1981 commented 4 years ago

I already saw changes reported at https://community.home-assistant.io/t/add-support-for-fujitsu-wireless-air-conditioning-control-app-fglair/65999/103 but I don't understand what contain file token.txt...

meichthys commented 4 years ago

@pilot1981 I'm at the same point as you. I believe the token.txt file can be left empty. I think it populates automatically when connecting to the FGLair server.

meichthys commented 4 years ago

@Mmodarre Could we get this pull request approved? We have a little work to do to get things working for v0.113, but this would be a good start.