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
Remove additional config steps required when running Fabric version on Android #1322
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me
README-Fabric.md
Outdated
1. Add latest react-native-screens | ||
2. on iOS | ||
- Install pods using `RCT_NEW_ARCH_ENABLED=1 pod install` (iOS only) | ||
- Install pods using `RCT_NEW_ARCH_ENABLED=1 pod install` – this is a ste |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be this is the same as in Fabric app
?
android/src/main/java/com/swmansion/rnscreens/RNScreensPackage.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 🚀
Just fix the typo in docs
Description
This PR simplifies the installation process on Android when using Fabric by removing the step when cpp dependency needs to be manually added to the main app's Android.mk file and also getting rid of the registry call that was ought to be added to ReactNativeHost implementation.
Changes
The following changes has been made:
Test code and steps to reproduce
Tested this in the following scenarios: