-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
docs: update broken react-native-reanimated links #1284
docs: update broken react-native-reanimated links #1284
Conversation
✅ Deploy Preview for react-navigation-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Thanks for the PR. Drawer supports Reanimated 3 as well as it's the same API as Reanimated 2.
Can you update the PR to link to the latest docs instead of version 2 specifically? Maybe also change Reanimated 2 to "the latest version of Reanimated"
@satya164 agree, done! |
@@ -38,7 +38,7 @@ Then, you need to install and configure the libraries that are required by the d | |||
npm install react-native-gesture-handler react-native-reanimated | |||
``` | |||
|
|||
The Drawer supports both Reanimated 1 and Reanimated 2. If you want to use Reanimated 2, make sure to configure it following the [installation guide](https://docs.swmansion.com/react-native-reanimated/docs/fundamentals/installation). | |||
The Drawer supports the latest version of Reanimated. Make sure to configure it following the [installation guide](https://docs.swmansion.com/react-native-reanimated/docs/fundamentals/getting-started). |
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.
This isn't quite correct. It supports both Reanimated 1 and the latest version. Reanimated 1 has a different API and supported via useLegacyImplementation
prop. The latest version of Reanimated needs additional configuration.
Basically what was written before, but Reanimated 2 should be replaced by latest version.
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.
@satya164 You're right, sorry for the confusion. Sent a fix, let me know if it's good now!
This PR fixes a few broken links pointing to react-naitve-reanimated 2 and 3.
In their new docs, v2 installation is now at
/docs/2.x/fundamentals/installation
, and v3 atdocs/fundamentals/getting-started