Add health connect snippets#915
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds several helper functions to the HealthConnectManager class for managing permissions and inserting various Health Connect records. The review identified several issues: an empty conditional block for permission checks, incorrect property usage in the PlannedExerciseSessionRecord constructor, unused function parameters, and typos in function names. Addressing these will improve code correctness, cleanliness, and maintainability.
yrezgui
left a comment
There was a problem hiding this comment.
You need to apply spotless
I have ran spotless command ( ./gradlew spotlessApply) |
| healthConnectClient: HealthConnectClient, | ||
| record: StepsRecord |
There was a problem hiding this comment.
how about this pending comment?
| return permissions | ||
| } | ||
|
|
||
| suspend fun createPermissionHeartRate(): Set<String> { |
There was a problem hiding this comment.
same here, what do you think of this pending comment?
Add snippets for Health Connect site