Skip to content

Conversation

@raphael-papazikas
Copy link

The current example code in the README is not based on the most recent version of entry.server.node.tsx

Some examples:
https://github.com/remix-run/react-router/blob/64f82f1581bd5e72a9312802c60bb61b9278135a/packages/react-router-dev/config/defaults/entry.server.node.tsx#L17-L19

https://github.com/remix-run/react-router/blob/64f82f1581bd5e72a9312802c60bb61b9278135a/packages/react-router-dev/config/defaults/entry.server.node.tsx#L34-L37

Updated the code from a full file to a patch because it's easier to understand the actual changes and apply accordingly based on the setup and version used.

Updated README to clarify entry-server.tsx configuration and code changes.
Comment on lines +393 to +395
- loadContext: AppLoadContext,
- // If you have middleware enabled:
- // loadContext: RouterContextProvider
Copy link
Owner

Choose a reason for hiding this comment

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

Middleware is required for remix-i18next so there's no scenario where the user may not have middleware enabled

Copy link
Author

Choose a reason for hiding this comment

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

Yes and therefore everything removed from the boilerplate

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