Skip to content

[Health] add delete by client id and fetch unit support #1223

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

connor-callaghan
Copy link

@connor-callaghan connor-callaghan commented Jun 25, 2025

This pull request focuses on unit conversions, and record deletion capabilities. The changes improve the flexibility of data handling and add support for more granular operations on health records.

Unit Conversion:

  • HealthDataConverter: Added support for unit conversions in WeightRecord, HeightRecord, BodyTemperatureRecord, and BloodGlucoseRecord by introducing a dataUnit parameter to the convertRecord method. This allows dynamic conversion based on specified units (e.g., kilograms to pounds, Celsius to Fahrenheit).

Metadata:

  • HealthDataWriter: Updated various methods (writeData, writeWorkoutData, writeBloodPressure, and writeMeal) to include optional clientRecordId and clientRecordVersion parameters in the Metadata object. This facilitates better tracking and versioning of health records. [1] [2] [3]

Record Deletion:

  • HealthDataOperations: Introduced a new deleteByClientRecordId method to allow precise deletion of health records using clientRecordId and dataTypeKey. This provides a more targeted approach to record management.

Data Retrieval:

  • HealthDataReader: Added support for specifying a dataUnit parameter during data retrieval, enabling unit-specific filtering and processing in the getData method. [1] [2]

@connor-callaghan connor-callaghan changed the title add delete by client id and fetch unit support [Health] add delete by client id and fetch unit support Jun 26, 2025
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.

2 participants