Drakulix / googlehome

Google Home Integration for Home-Assistant
Apache License 2.0
29 stars 3 forks source link

Additional option entry for prefetched master tokens #16

Closed arunoruto closed 3 years ago

arunoruto commented 3 years ago

Bypass the retrieval of the master token by providing it at the setup process. The password field is still required, but is not actively used afterwards.

Drakulix commented 3 years ago

Could you edit the description to explain to the user, that the master token is optional? (And add (Optional) to the end of the "master token" string?)

For most users providing a password is likely way easier and "master token" has no meaning for them. Otherwise I am willing to provide this workaround until a better solution for #14 is found.

arunoruto commented 3 years ago

I hope the addition to the description is okay now. I'm also checking if the provided master token is correct by fetching an access token and looking at the result. Hence I also created a new error to be thrown.

Drakulix commented 3 years ago

Thanks!

Edo78 commented 3 years ago

I installed version 0.4.0 but the new field is without any label image