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

fix: use async_create for notification #2380

Merged
merged 7 commits into from
Aug 2, 2024
Merged

fix: use async_create for notification #2380

merged 7 commits into from
Aug 2, 2024

Conversation

ChristophCaina
Copy link
Contributor

fixed issue with partition cookies
(https://gitlab.com/keatontaylor/alexapy/-/commit/e594a3b194eec7d6451ad0c7135d8c32704f5f73)

unfortunately, this change does not solve the issue #2378

@alandtse
Copy link
Owner

alandtse commented Aug 1, 2024

Thanks. However a dependency bump requires changes in multiple places. Please take a look at past bumps to see what is required.

@ChristophCaina
Copy link
Contributor Author

ChristophCaina commented Aug 1, 2024

Thanks,
I will Check this.

  • changed pyproject.tmol - but I don't have an environment to auto-generate the poetry file.

ChristophCaina and others added 3 commits August 1, 2024 19:47
fix depricated hass.components.persistent_notification.async_create
@ChristophCaina
Copy link
Contributor Author

should now also fix #2382 - but since my installation is not loading, I cannot test this further :-(

@alandtse
Copy link
Owner

alandtse commented Aug 1, 2024

Since the alexapy bump isn't complete, let's just do the persistent notification with this PR.

@ChristophCaina
Copy link
Contributor Author

reverted the version bump back :)

@alandtse alandtse changed the title bump alexapy to 1.28.1 fix: use async_create for notification Aug 2, 2024
@alandtse alandtse merged commit 48532e4 into alandtse:dev Aug 2, 2024
2 of 3 checks passed
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