Healthfit with Android Override - #4690
Merged
Merged
Conversation
Owner
Author
|
it should close the #3613 |
Owner
Author
|
depends from #4605 |
…ce write - Replace HealthConnectHelper.java stub with full Kotlin implementation - Add HealthConnectPermissionActivity.kt for permission request flow - Wire healthConnectWriteWorkout() in homeform.cpp fit_save_clicked() — this was the root cause of no activity being created on workout stop - Power (PowerRecord) and cadence (CyclingPedalingCadenceRecord) are now written for all device types that emit non-zero values - Add Kotlin plugin, activity-ktx, and Health Connect write permissions to build.gradle and AndroidManifest.xml (compileSdk bumped to 35) - Add 20 local JUnit tests covering exerciseType mapping, buildRecords logic, power/cadence inclusion, and edge cases Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… introduced by rebase Removes 4 duplicate lines in build.gradle (kotlin classpath, apply plugin, activity-ktx dependency, kotlin.srcDirs) and 2 duplicate blocks in AndroidManifest.xml (second ViewPermissionUsageActivity alias targeting CustomQtActivity, duplicate HealthConnectPermissionActivity without intent-filter) that were left from conflict resolution during rebase. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tial permissions - Replace one-shot SHOWN guard with iOS-like behaviour: every app start, check missing permissions and show the dialog if any are absent (guarded only by PENDING flag to avoid double-showing) - Remove PREF_PERMISSION_PROMPT_SHOWN constant (no longer used) - writeWorkoutJson now only requires WRITE_EXERCISE; all other permissions (distance, calories, HR, power, speed, cadence) are optional — records for ungranted permissions are silently skipped instead of aborting the whole write - buildRecords accepts an optional grantedPermissions set (default = all) so existing unit tests are unaffected Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Automatic update of translatable strings extracted from source code. Updated 30 language files in src/translations/ - Updated by GitHub Actions (Testing Mode) - Triggered by: pull_request - Date: 2026-06-08 13:37:16 UTC
…myos-zwift into healthfit-override
… records Add ElevationGainedRecord, StepsRecord, HeartRateVariabilityRmssdRecord and StepsCadenceRecord to Health Connect export. Treadmill cadence now uses StepsCadenceRecord (steps/min) instead of CyclingPedalingCadenceRecord. Update AndroidManifest.xml with the four new write permissions and extend unit tests to cover all new record types. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Automatic update of translatable strings extracted from source code. Updated 30 language files in src/translations/ - Updated by GitHub Actions (Testing Mode) - Triggered by: pull_request - Date: 2026-06-08 16:12:40 UTC
Switch ExerciseSessionRecord metadata from RECORDING_METHOD_MANUAL_ENTRY to RECORDING_METHOD_ACTIVELY_RECORDED so Health Connect displays the app name (QZ) as the data source, matching how Peloton and other apps appear. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.