fix: Android TurboModule compatibility with RN 0.80 #15
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.
Overview
Fixes a TurboModule compatibility issue for Android.
Starting from RN 0.80,
isBlockingSynchronousMethod = true
annotated methods can't bevoid
and they must return a value.Fixes: GetStream/stream-video-js#1858
As part of this PR, we are also upgrading the sample apps to RN 0.80.1 and enabling
new architecture
by default for them.Ref: https://github.com/facebook/react-native/blob/v0.80.1/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/turbomodule/core/TurboModuleInteropUtils.kt#L59