search

matinzd / react-native-health-connect

React native library for health connect (Android only)
http://matinzd.github.io/react-native-health-connect
MIT License
207 stars 43 forks source link

revokeAllPermissions in not work on Android #104

Open CohesionJamesTsao opened 3 months ago

CohesionJamesTsao commented 3 months ago

I followed all the settings in the document and other functions can be used normally. But only the revokeAllPermissions function has no effect in Android 14 But in Android 13, it has normal removal permissions.

Environment:

import { initialize, requestPermission, getGrantedPermissions, readRecords, revokeAllPermissions, getSdkStatus, SdkAvailabilityStatus } from 'react-native-health-connect';

`

revokePermissions 
      </TouchableOpacity>`
nestorLanex2 commented 3 months ago

I noticed this as well, What happened on my end though is that when i close the app after revoking app permissions, the permissions on the health connect were revoked. but only after I closed the app

If i don't close the app (not only simply putting it on the background), the permission changes don't seem to reflect

bwmannering commented 3 months ago

Same here - couldn't get this function to work

matinzd commented 3 months ago

This function hasn't been changed for a while and it can be due to changes from Google. So, expect these behaviours in alpha version. I will try to narrow down the issue when I find some time.

matinzd commented 2 months ago

Finally I got some time to check this out!

This can be related to permission delegate API not being fully compatible with RN. Can everyone verify closing and opening the app would show the changes?

ugurakin1 commented 1 month ago

Same here - I don't get the changes applied even on app close/re-open.

EDIT: I've tried again today and now I was able to revoke permissions after app close/re-open. Did a wipe emulator/clean/sync/build maybe that impacted things.

EDIT x2: Tested again after regressing the set of permissions my app requests and the call works without app close/re-open. I'm highly suspicious that adding all the permissions the app needed, which is a lot of permissions, may have broken all kinds of things - including this. Initially, I was testing with just R|W Exercise permissions and everything was working well. All hell broke loose once I added more permissions. I'm going to focus on continuing with a limited set of permissions for now and slowly introduce new permissions later on. Will update if I find out more.

BlocknBoom123 commented 2 weeks ago

Finally I got some time to check this out!

This can be related to permission delegate API not being fully compatible with RN. Can everyone verify closing and opening the app would show the changes?

Please let us know if revokePermissions works now or what we should do as I have to close the entire app and reopen it for permissions to be taken away. Thank you.