-
Notifications
You must be signed in to change notification settings - Fork 9
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
[0.78] Relax TurboModule interop layer strictness to prevent app crashes #773
Labels
Type Pick Request
Pick requests to include commits inside a React Native release
Comments
Thank you for opening a new pick-request for React Native. If your pick does not satisfy the criteria below, please close it as it will not be considered. ✅ Which pick requests we accept
❌ Which pick requests we don’t accept
ℹ️ What makes for a good pickIn order for your pick to be considered, please do the following:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Target Branch(es)
0.78
Link to commit or PR to be picked
facebook/react-native@3bd3f10
Description
The interop layer for TM validates the method signatures as soon as a legacy module is loaded.
There are some scenarios, especially when a legacy module is implemented in Swift, where the signatures in objective-c and swift does not match and that would make the app crash because the runtime can't find the exact signature we expect.
The old architecture was not crashing but just bailing out.
This commit aligns the New Architecture behavior with the Old Architecture and it will unblock some people to migrate to the new architecture.
The text was updated successfully, but these errors were encountered: