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.

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

Fixup breaking changes on 2025.1, Async issues and current deprecations #31

Open
wants to merge 2 commits into
base: hacs
Choose a base branch
from

Conversation

geniesis
Copy link

@geniesis geniesis commented Jan 7, 2025

Just enough fixes code to enable use in 2025.1 as raised in #30

Two main issues causing the failure:

  1. Use of async_forward_entry_setup was removed in 2025.1
  2. cognito.authenticate() contains blocking IO calls. Traced to aioboto3.
  3. Also cleaned up use of HomeAssistantType which will be deprecated soon.

This is just enough to get it going on my setup. I have not fully tested all possible commands, what I have tested works so far.
Current setup has Evap+Heater no Zones

@roelfdiedericks
Copy link

Yay, thank you so much!

@roelfdiedericks
Copy link

I can confirm that this PR works post my 2025.1 upgrade. Props to @geniesis for the commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants