-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
RCTAsyncStorage.getAllKey callback does not happen RN 0.31 Android 4.2.2 when debugging #9319
Comments
I have replicated this issue on 0.31 and 0.32.0-rc.0. I can confirm that EDIT: Replicated on Android 6.0.1 with a Oneplus One and Nexus 9, so don't think it's related to your version of Android. |
I'm also experiencing this issue. It means I haven't been able to use the chrome debugger since upgrading to 0.31. Anyone have any workarounds for now? |
@alenia nope, still broken for me. Haven't had a chance to investigate. |
I have the same issue, it works as expected in 0.30. I have downgraded until the cause is determined. I have tested on a Nexus 5 a 6.0.1, and a Nexus 6P at N preview 5 |
I hit this as well in 0.32.0-rc. I suspect that the commit in 0.31.0 (c89c25f) which makes the new bridge the default bridge introduced the issue. |
@rigdern the bridge is definitely the issue. I modified my MainApplication to call I don't have the time to debug this issue fully beyond that, but for those experiencing this issue, I posted a workaround: https://gist.github.com/kageurufu/79e4aaa1c41cbdb147e936d7e473029d |
@rigdern Thanks for looking into it! I'll ping @cjhopman and @mhorowitz on fb@work messenger. |
Closing in favor of #9436 |
Android 4.2.2
RN 0.31
RCTAsyncStorage.getAllKeys(function(error, keys) {
line inAsyncStorage.js
,getAllKeys
method never gets to callback when debugging in chrome.Also,
AsyncStorage.js
chooses to use sqlite implementation (at least in the chrome debugger).This issue happen only in debug.
The text was updated successfully, but these errors were encountered: