Skip to content

Fix deprecation: import persistent_notification and use it directly#7

Open
bobrippling wants to merge 1 commit into
rarosalion:mainfrom
bobrippling:fix/notify-deprecation
Open

Fix deprecation: import persistent_notification and use it directly#7
bobrippling wants to merge 1 commit into
rarosalion:mainfrom
bobrippling:fix/notify-deprecation

Conversation

@bobrippling

@bobrippling bobrippling commented Feb 20, 2025

Copy link
Copy Markdown

See also home-assistant/core#63901. This avoids the below deprecation:

Detected that custom integration 'authenticated' accesses hass.components.persistent_notification which should be updated to import functions used from persistent_notification directly at custom_components/authenticated/sensor.py, line XYZ: hass.components.persistent_notification.create(
This will stop working in Home Assistant 2025.3, please report it to the author of the 'authenticated' custom integration

Tested: login notifications still appear as expected

@bobrippling
bobrippling force-pushed the fix/notify-deprecation branch from 3a1d21a to 7b13f82 Compare February 24, 2025 12:23
@bobrippling
bobrippling marked this pull request as draft February 24, 2025 18:04
See also home-assistant/core#63901. This avoids the below deprecation:

```
Detected that custom integration 'authenticated' accesses hass.components.persistent_notification which should be updated to import functions used from persistent_notification directly at custom_components/authenticated/sensor.py, line XYZ: hass.components.persistent_notification.create(
This will stop working in Home Assistant 2025.3, please report it to the author of the 'authenticated' custom integration
```
@bobrippling
bobrippling force-pushed the fix/notify-deprecation branch from 7b13f82 to c7fa1ca Compare February 24, 2025 22:00
@bobrippling
bobrippling marked this pull request as ready for review February 25, 2025 12:05
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