-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
1.13 - 1.14 : @remix-run/server-runtime EntryContext is not assignable to type @remix-run/react EntryContext 🔴 #5615
Comments
Same does not repro for me with 1.14 when I write import { RemixServer } from "@remix-run/react";
import type { EntryContext } from "@remix-run/node";
let context: EntryContext;
export const SomeServer = () => <RemixServer url='some-url' context={context} />; What is the error you are getting @raskyer ? |
I should add that this happend to me since I upgraded from 1.11 |
What do you mean ? You repro or not ? I have the exact type error in screenshot I have posted. Upgrading to 1.14 does not solve the issue. I have remove my node_modules / package-lock but it doesn't solve the issue. It's a typescript error, not node or runtime. |
i have created a thread here: https://discord.com/channels/770287896669978684/1082941041306505217 |
|
@raskyer it is the very same error as I have |
@alexhoo Yes Alex :) I think every one with new typescript / remix project will have it. |
what version of React are you using @raskyer ? |
17.0.2 |
Ok, I think I found out why. @remix-run/server-runtime use @types/react version 18.0.15 but locally I have version 17.0.53. One fix is to update @types/react to stick to the version used by @remix-run/server-runtime. |
have you tried? |
Now the question we should ask is : Is it normal that Maybe @machour or @brophdawg11 have an idea ? |
This is fixed by #5713 and should be available in the next Remix release. We ended up removing |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
Hm, we must have a bug in our bot logic - this is not included in https://github.com/remix-run/remix/compare/remix@1.17.0...remix@1.17.1-pre.1 |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
How exactly should this be fixed?
|
Same error in 2.* |
Please open a new issue with a minimal reproduction and we can take a look |
I've found the error by the way. It was not a remix issue in my exact case mixed with i18next usage.
|
What version of Remix are you using?
1.13.0
Are all your remix dependencies & dev-dependencies using the same version?
Steps to Reproduce
Install remix server template.
Typescript version : 4.9.5
Expected Behavior
RemixServer doesn't complain with remixContext
Actual Behavior
Type error with remix context.
The text was updated successfully, but these errors were encountered: