-
Notifications
You must be signed in to change notification settings - Fork 656
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
[health] Fixed request permission to Health Connect for Android #927
[health] Fixed request permission to Health Connect for Android #927
Conversation
Hi @iamLuCat - I really like this PR, but I've been updating the Health plugin to version 10.0.0 (published), and this PR would need to go into this version. I can't quite see exactly what changes this PR adds and it would be best if you could update this PR to be compatible and mergeable with the current master branch. |
Differences:
|
@bardram I will update the version today. Thank you for your response. |
7d3978e
to
67e73e2
Compare
@iamLuCat - please see the comment from @ciriousjoker above. As far as I can see, this PR can be replaced by #941 and #943. So - there is no need to do anything here. Right? |
@bardram That's correct. I want to make a slight adjustment on my branch. @ciriousjoker's PR is better. |
This has now been released in 10.2.0 |
Issue: #893
Fix the issue of requesting permission when the Health Connect application hasn't been installed
Video demo
media_20240327_140219_2556024780277294492.mp4