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

Update on the usage of deprecated Vacuum constants & module service import. #1914

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

estebanz01
Copy link

This fixes the following:

  • VacuumActivity.RETURNING instead of STATE_RETURNING
  • VacuumActivity.PAUSED instead of STATE_PAUSED`
  • VacuumActivity.IDLE instead of STATE_IDLE
  • VacuumActivity.ERROR instead of STATE_ERROR`
  • VacuumActivity.DOCKED instead of STATE_DOCKED
  • VacuumActivity.CLEANING instead of STATE_CLEANING
  • from homeassistant.helpers.service import async_register_admin_service instead of hass.helpers.service.async_register_admin_service.

Based on https://developers.home-assistant.io/blog/2024/06/12/async_forward_entry_setups/
it is possible to replace it with `async_forward_entry_setups` instead,
which accepts multiple platforms and it should be more efficient.
This change is needed as the deprecated constants will be removed in
version 2026.1.
This change removes the deprecated call to `hass.helpers.service` to use
the recommended function import, so it works on 2026.1.
@estebanz01 estebanz01 marked this pull request as draft January 23, 2025 17:44
@estebanz01 estebanz01 marked this pull request as ready for review January 23, 2025 17:49
@estebanz01
Copy link
Author

I just noticed that part of my work was made before here: #1810 so I'm more than happy to rebase after that PR gets merged.

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.

1 participant