Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Changing temperature gets overwritten by thermostat #10

Closed
PerfectlyNormal opened this issue Jan 29, 2022 · 0 comments · Fixed by #12
Closed

Changing temperature gets overwritten by thermostat #10

PerfectlyNormal opened this issue Jan 29, 2022 · 0 comments · Fixed by #12

Comments

@PerfectlyNormal
Copy link
Collaborator

Hi, done some testing now and it looks like the EB500 dont want to update changes done from HA to thermostat/APP but changes in app updates HA after some minutes. The power sensor seems to just read the installed effect it dosen reflect if the heat is on or off. The energy sensor works as expected. :-)

Same here regarding changing the temperature from HA. It just gets overwritten by the termostats set value some seconds later and same for me regarding the effect.
@PerfectlyNormal could you put some time to try to fix it?

Originally posted by @joggs in #8 (comment)
@PerfectlyNormal PerfectlyNormal mentioned this issue Jan 29, 2022
Merged
PerfectlyNormal added a commit to PerfectlyNormal/home_assistant_ebeco that referenced this issue Jan 29, 2022
@PerfectlyNormal
Fix extra device_id being passed to data handler
83af15c 
This fixes joggs#10.
PerfectlyNormal added a commit to PerfectlyNormal/home_assistant_ebeco that referenced this issue Jan 29, 2022
@PerfectlyNormal
Don't await non-async method to update coordinator data
bd4b808 
Can't trust the prefix `async_`, gotta check that the
method actually is async. joggs#10
@PerfectlyNormal PerfectlyNormal mentioned this issue Jan 29, 2022
Merged
@joggs joggs closed this as completed in #12 Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix updating target temperature from integration PerfectlyNormal/home_assistant_ebeco
1 participant
@PerfectlyNormal