Skip to content

Conversation

@jpelgrom
Copy link
Member

@jpelgrom jpelgrom commented Oct 22, 2025

Summary

Fixes #5948, hopefully

When using the system's Health Connect implementation you cannot update Health Connect, so add a check for the background read permission to only request it if supported. Also see: #5510

Checklist

  • New or updated tests have been added to cover the changes following the testing guidelines. - I tried adding one, please let me know if it's adequate.
  • The code follows the project's code style and best_practices.
  • The changes have been thoroughly tested, and edge cases have been considered.
  • Changes are backward compatible whenever feasible. Any breaking changes are documented in the changelog for users and/or in the code for developers depending on the relevance.

Screenshots

n/a

Link to pull request in documentation repositories

n/a

Any other notes

@jpelgrom jpelgrom changed the title Check for background feature availability in Health Connect Check for background read feature availability in Health Connect Oct 22, 2025
@TimoPtr TimoPtr self-requested a review October 24, 2025 07:02
@TimoPtr TimoPtr marked this pull request as ready for review October 24, 2025 07:02
@TimoPtr TimoPtr merged commit 1f90fce into home-assistant:main Oct 24, 2025
28 of 29 checks passed
@jpelgrom jpelgrom deleted the health-connect-permission-featurecheck branch October 24, 2025 07:07
@jpelgrom
Copy link
Member Author

jpelgrom commented Oct 24, 2025

User confirmed this fixes the issue so good to merge. @TimoPtr You beat me to marking it as ready :)

@dshokouhi #5612 will need adjustments because of this, but this PR's change should also make it work on older API versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Health Connect permission issue on /e/ OS

2 participants