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

Use _unrecorded_attributes instead of recorder_history_prefilter #121

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

Snuffy2
Copy link

@Snuffy2 Snuffy2 commented Jul 4, 2024

Uses the new _unrecorded_attributes = frozenset({MATCH_ALL}) functionality available with HA 2024.7 instead of recorder_history_prefilter
https://developers.home-assistant.io/blog/2024/06/22/excluding-state-attributes-from-recording-match-all/
https://developers.home-assistant.io/docs/core/entity#excluding-state-attributes-from-recorder-history

Fixes #120

It would be great if someone who is having the issues could test this as well.

This may not be the best long term solution but should work for now pending: home-assistant/architecture#1106

@Snuffy2 Snuffy2 force-pushed the Fix-recorder-errors-in-2024.7 branch 2 times, most recently from 66ffd53 to 5338b3b Compare July 4, 2024 22:30
@Snuffy2 Snuffy2 marked this pull request as ready for review July 4, 2024 22:48
@Snuffy2
Copy link
Author

Snuffy2 commented Jul 5, 2024

@enkama unfortunately, the recorder_history_prefilter (which is in use by hass-variables) is breaking things for many users in 2024.7. If you can get this PR out into a release quickly it would be helpful.

@enkama enkama merged commit dd62d7a into enkama:master Jul 5, 2024
1 of 2 checks passed
@Snuffy2
Copy link
Author

Snuffy2 commented Jul 5, 2024

Thanks @enkama !!!

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.

Error running job after upgrade HA to 2024.7
2 participants