-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Optional TemperatureClient to redirect temperature readings another HA entity #22
Conversation
* Added the option to quickly reload it from dev tools * Update climate.py replaced deprecated consts * Update climate.py * Update climate.py Remove long text from log * Add asyncio * Make unit tests pass * Remove custom_components climate.py * Bump version number --------- Co-authored-by: Dan Timu <[email protected]> Co-authored-by: aver-ua <[email protected]> Co-authored-by: floringhimie <[email protected]>
* Added the option to quickly reload it from dev tools * Update climate.py replaced deprecated consts * Update climate.py * Update climate.py Remove long text from log * Add asyncio * Make unit tests pass * Remove custom_components climate.py * Bump version number --------- Co-authored-by: Dan Timu <[email protected]> Co-authored-by: aver-ua <[email protected]> Co-authored-by: floringhimie <[email protected]>
I have installed your component and On run I have the following log message: climate:
- platform: salusfy
username: "email"
password: "password"
id: "salusDeviceId"
enable_temperature_client: False And it looks like the integration is looking for To make the integration work, I had to add this into my config: climate:
- platform: salusfy
access_token: "your-HA-access-token"
entity_id: "your-entity"
username: "email"
password: "password"
id: "salusDeviceId"
enable_temperature_client: False |
Ah thanks, I hadn't tested without those values. I've made them optional now. What was the log message you got? |
I will try to fork and make a HACS compatible component from your fork with am instalation UI, in order to make the component more flexible and no pass passwords through config files. |
I've just removed the extra settings from configuration.yaml and confirmed it now works correctly. |
Fixes #17, #19
I've been working on a fork for a while. I'm not sure if any of the changes are of interest but there may be some useful bits.
TemperatureClient
andClient
wrapper to redirect temperature readings to another entity via the Home Assistant API