Add Health connect snippets#866
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces the healthconnect module, providing integration with the Health Connect API. Key additions include managers for handling data changes and records, a background worker for periodic reads, and a companion service for wearable devices. Feedback focuses on correcting a package name error in the theme configuration, removing duplicate dependencies and unused imports, and cleaning up unused constructor parameters in the manager classes. Additionally, a misleading comment regarding an early return in the UI logic should be clarified.
kkuan2011
left a comment
There was a problem hiding this comment.
I ran the GitHub workflows, and it looks like the spotless check failed as well as the build. I have a suggestion for fixing the build, but not sure about the spotless check error. I wonder if you have to squash all your commits into 1 commit, and then force push the branch again, so that there's only 1 commit on top of main?
Add Health Connect snippets to main.