You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We're moving RNFB API to match Firebase web JS SDK API, some API are currently asynchronous when they should be synchronous to match Firebase web JS SDK API.
languageCode and tenantId are synchronous getters/setters.
This is an issue to track this work, and to help developers find an alternative solution to the deprecated JS debugger which has an issue with synchronous native calls.
examples of how to make a native method synchronous, at least in old architecture ("Paper") react-native style (unsure if it is possible in Fabric / TurboModules)
Android:
Add annotation argument to @ReactMethod call like so:
Both of those break the ability to use the old react-native javascript debugging interface as that pulls the bundle across the network to the debugger, executes it in the debugger context and sends the results back and forth to the native app, something that can't happen synchronously.
Historically the answer to this is "use reactotron" for debugging, but I believe react-native now has wholly moved to "React Native DevTools" as official recommendation
Hello 👋, to help manage issues we automatically close stale issues.
This issue has been automatically marked as stale because it has not had activity for quite some time.Has this issue been fixed, or does it still require attention?
This issue will be closed in 15 days if no further activity occurs.
We're moving RNFB API to match Firebase web JS SDK API, some API are currently asynchronous when they should be synchronous to match Firebase web JS SDK API.
For example, Auth: https://firebase.google.com/docs/reference/js/auth.auth
languageCode
andtenantId
are synchronous getters/setters.This is an issue to track this work, and to help developers find an alternative solution to the deprecated JS debugger which has an issue with synchronous native calls.
@mikehardy - for visibility.
The text was updated successfully, but these errors were encountered: