Skip to content
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

Merged

Conversation

thiagobrez
Copy link
Contributor

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 at docs/fundamentals/getting-started

Copy link

netlify bot commented Nov 5, 2023

Deploy Preview for react-navigation-docs ready!

Name Link
🔨 Latest commit 751dffc
🔍 Latest deploy log https://app.netlify.com/sites/react-navigation-docs/deploys/65496b5c10bb0c0008b28cf3
😎 Deploy Preview https://deploy-preview-1284--react-navigation-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@satya164 satya164 left a 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"

@thiagobrez
Copy link
Contributor Author

@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).
Copy link
Member

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.

Copy link
Contributor Author

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!

versioned_docs/version-6.x/drawer-layout.md Outdated Show resolved Hide resolved
versioned_docs/version-6.x/drawer-navigator.md Outdated Show resolved Hide resolved
versioned_docs/version-7.x/drawer-layout.md Outdated Show resolved Hide resolved
versioned_docs/version-7.x/drawer-navigator.md Outdated Show resolved Hide resolved
@satya164 satya164 enabled auto-merge (rebase) November 6, 2023 22:41
@satya164 satya164 merged commit 792efa4 into react-navigation:main Nov 6, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants