Added "aggregate" function to get statistics about metrics. #18
Conversation
This file contains 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
|
@imdzx Should I assign that to someone for review? Let me know if I can help. |
|
@quentinleguennec can you Please add the functions for getting Health Connect aggregate data into buckets? There are two types of buckets you can use duration (AggregateGroupByDurationRequest) and period(AggregateGroupByPeriodRequest). |
quentinleguennec
force-pushed
the
feature_added_aggregate_function
branch
from
2080ab5 to
cf62684
Compare
quentinleguennec
force-pushed
the
feature_added_aggregate_function
branch
from
cf62684 to
da03ad8
Compare
Added support for the Privacy Policy button redirect in the Google Health permission Activity.
Updated README.md. Ran AGP upgrade assistant.
quentinleguennec
force-pushed
the
feature_added_aggregate_function
branch
3 times, most recently
from
895595b to
961c3e1
Compare
…ped from within the permission activity. Updated README.md. Updated example app.
quentinleguennec
force-pushed
the
feature_added_aggregate_function
branch
from
961c3e1 to
a177525
Compare
…talled. Improved errors handling. Refactored code.
…ature_added_aggregate_function' into feature_added_aggregate_function-merge # Conflicts: # android/src/main/kotlin/dev/duynp/flutter_health_connect/FlutterHealthConnectPlugin.kt # lib/src/flutter_health_connect_factory.dart
BREAKING: `writeData` now returns a list of the uids of the created Records instead of a boolean. Fixed `deleteRecordsByIds`, which would delete the records but then throw an error. Fixed `deleteRecordsByTime`, which would delete the records but then throw an error. Added documentation. Improved the Example app to add a button for each `deleteRecordsByIds` and `deleteRecordsByTime`.
Fixed issue with timestamps: [#1](#1) by [AliakseiT](https://github.com/AliakseiT) and [nikolaydymura](https://github.com/nikolaydymura) Added `getRecordById` [#1](#1) by [AliakseiT](https://github.com/AliakseiT) and [nikolaydymura](https://github.com/nikolaydymura) Improved the Example app to add a button for `getRecordById`.
…tion Added exercise route fetching and fixed time zones parsing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Hi, this pull request adds a new function to get aggregated statistics about metrics (using the android function "aggregate").
I also updated the README to mention:
This was preventing me from compiling when I tested on my app.