Skip to content

Conversation

@okaufmann
Copy link
Contributor

Related: #62

Problem

  • Home Assistant warned that async_config_entry_first_refresh is only supported when the coordinator is bound to a config entry.

Change

  • Update all coordinators to accept an optional config_entry and pass it to the base DataUpdateCoordinator.
  • Wire config_entry from async_setup_entry when constructing coordinators.

Impact

  • Removes deprecation warnings in HA logs.
  • No behavior change for data fetching/writing.

Testing

  • Manual: reload integration and confirm no deprecation warnings present.

…tors and setup to fix HA deprecation warnings
@okaufmann okaufmann changed the title askoheat: pass config_entry to DataUpdateCoordinator; fix deprecation warnings pass config_entry to DataUpdateCoordinator; fix deprecation warnings Aug 19, 2025
@toggm
Copy link
Owner

toggm commented Aug 20, 2025

Thanks!

@toggm
Copy link
Owner

toggm commented Aug 20, 2025

@okaufmann You need to:

  1. fix the linter issues ( python3 -m ruff check . ) and possibly reformat the files (python3 -m ruff format)
  2. fix the integration in the tests so they success (python3 -m pip install -r requirements.test.txt && python3 -m pytest)

Let me know if you need help.

@toggm
Copy link
Owner

toggm commented Sep 25, 2025

@okaufmann Any progress on this or do you need help?

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