Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
should we just add a dynamic redirect? |
|
ahh, it's throwing errors because it thinks the destination paths don't exist 🤔 we can probably update that logic to check if the destination path includes |
There was a problem hiding this comment.
@alexisintech Not sure if we should remove from the schema too?
There was a problem hiding this comment.
i left it because i wanted nick to weigh in!
There was a problem hiding this comment.
yes feel free to remove it from here
There was a problem hiding this comment.
And would be good to remove from scripts/lib/schemas.ts too
| }, | ||
| { | ||
| "source": "/docs/reference/remix/root-auth-loader", | ||
| "destination": "/docs/core-2/remix/reference/remix/root-auth-loader" | ||
| }, | ||
| { | ||
| "source": "/docs/reference/remix/overview", | ||
| "destination": "/docs/core-2/remix/reference/remix/overview" | ||
| }, | ||
| { | ||
| "source": "/docs/reference/remix/clerk-app", | ||
| "destination": "/docs/core-2/remix/reference/remix/clerk-app" | ||
| }, |
There was a problem hiding this comment.
@alexisintech These can be replaced with a dynamic redirect potentially:
{
"source": "/docs/reference/remix/:path*",
"destination": "/docs/core-2/reference/remix/:path*",
"permanent": true,
}
There was a problem hiding this comment.
Whether we make it a dynamic or static redirect, I don't think the destination should have "remix" twice?
E.g: "destination": "/docs/core-2/remix/reference/remix/root-auth-loader" should be "destination": "/docs/core-2/reference/remix/root-auth-loader".
There was a problem hiding this comment.
Whether we make it a dynamic or static redirect, I don't think the destination should have "remix" twice?
E.g:
"destination": "/docs/core-2/remix/reference/remix/root-auth-loader"should be"destination": "/docs/core-2/reference/remix/root-auth-loader".
oh yeah can fix those ones
There was a problem hiding this comment.
I would prefer we used static redirects over dynamic, but thats mainly cuz I want to simplify the redirects to only being static, getting rid of dynamic redirects all together.
|
@alexisintech Have reviewed & left some comments! There is sth weird happening w/ some of the redirects for sure - especially with the "clerk-app" page. For some reason, even tho the destination says Screen.Recording.2026-03-09.at.11.03.12.am.movOther than this, we'll have to make sure to have a sibling |
|
okay just fixed all the redirects and tested them and they're working - the only one not working is |
@NWylynko if we remove it from |
Hmm weird, it's working locally. But yeah not on the preview link? @NWylynko @alexisintech Screen.Recording.2026-03-09.at.11.43.28.am.movSeems like we need to update the tests too to remove Remix from the failing "Test build". |
I think we will want to start with creating a clerk/clerk pr that removes remix from the available sdks and adds to the migration map https://github.com/clerk/clerk/pull/2217 should be good |
oh it wasn't even working locally for me LOL |
Tbh, the redirects don't seem to work for me on the previews. This might be an issue with the preview link itself potentially because locally, it works.. Screen.Recording.2026-03-09.at.11.58.31.am.mov |
|
@NWylynko could you help with the tests? I'm confused on why it's failing - i simply switched remix for expo?
|
All fixed here: Fix tests. Was mostly due to places where "remix" was replaced with "react" rather than "expo", but also the output needing fixing. |
|
marking as blocked until https://github.com/clerk/clerk/pull/2217 is merged |

What does this solve? What changed?
The Remix SDK has been in maintenance mode, as Remix is a no longer supported framework and defers to React Router.
This PR serves to remove Remix from the docs, and adds redirects to point users to /docs/core-2 for any Remix needs.
Deadline
Related
clerk/clerk-docsinitial PR: Remove clear references to Remix.has been reverted https://github.com/clerk/clerk/pull/2245clerk/clerkPR: Docs - Remove remix, migrate to react routerSlack conversation: https://clerkinc.slack.com/archives/C01QFRQNHSN/p1772829651962499