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

fix: Update React Router mentions #1748

Merged
merged 1 commit into from
Nov 27, 2024
Merged

fix: Update React Router mentions #1748

merged 1 commit into from
Nov 27, 2024

Conversation

LekoArts
Copy link
Member

@LekoArts LekoArts commented Nov 27, 2024

Important

🔎 Previews:

Explanation:

  • React Router released a new major version (v7) and things have changes quite a bit!
  • So much that we'll release a new SDK for it (WIP PR: feat: Add react-router SDK javascript#4621) but in the interim we can update individual snippets and add a notice

Checklist

  • I have clicked on "Files changed" and performed a thorough self-review
  • I have added the "deploy-preview" label and added the preview link(s) to this PR description
  • All existing checks pass

@LekoArts LekoArts requested a review from a team as a code owner November 27, 2024 11:35
Copy link

Hey, here’s your docs preview: https://clerk.com/docs/pr/1748

@@ -87,7 +87,7 @@ The `<SignedIn>` component offers authentication checks as a cross-cutting conce
Below is an example of how to use `<SignedIn>` with React Router. The `<SignedIn>` component can be used as a child of a `<Route />` component to render content only to signed in users.

```tsx {{ filename: 'app.tsx' }}
import { Routes, Route } from 'react-router-dom'
import { Routes, Route } from 'react-router'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -32,7 +32,7 @@ description: Learn how to add React Router to your React application using their
</TutorialHero>

> [!WARNING]
> [Loaders](https://reactrouter.com/en/main/route/loader) and [Actions](https://reactrouter.com/en/main/route/action) are currently not supported by Clerk with React Router.
> This tutorial is written for [React Router 6](https://reactrouter.com/6.28.0/home) and doesn't support [loaders](https://reactrouter.com/6.28.0/route/loader) and [actions](https://reactrouter.com/6.28.0/route/action).
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once the new SDK is out I'll revisit this doc

Copy link
Member

@alexisintech alexisintech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! thank you lennart :)

@alexisintech alexisintech merged commit ac79d69 into main Nov 27, 2024
4 checks passed
@alexisintech alexisintech deleted the rr6-notices branch November 27, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants