Skip to content

Conversation

brillout
Copy link
Contributor

No description provided.

@brillout
Copy link
Contributor Author

@arnaudsm I'm trying react-router-serve ./build/server/index.js but it fails, it seems like a React Router bug? How can I preview the production build?

@brillout brillout changed the title wip wip (vikejs/vike#212) Sep 17, 2025
@brillout brillout changed the title wip (vikejs/vike#212) wip (brillout/telefunc#212) Sep 17, 2025
@arnaudsm
Copy link
Owner

arnaudsm commented Sep 18, 2025

Found a workaround : react-router-serve ./build/server/index2.js works perfectly !
During the build, the telefunc bundle replaces server/index.js and the react-router bundle goes to server/index2.js instead.

This is hacky but it works. I hope it doesn't break any other react-router behavior, but it seems to be fine from my testing.
Do you want to implement a simpler fix in the telefunc/vite plugin ?

If not, I can propose a PR to update your site documentation for Remix about React-Router.

@brillout
Copy link
Contributor Author

I've an idea how to workaround it, let me have a look.

@brillout
Copy link
Contributor Author

a PR to update your site documentation for Remix about React-Router.

That would be great! Let's focus first on the build entry conflict and having a neat example.

@arnaudsm arnaudsm changed the title wip (brillout/telefunc#212) Fix Build Command Sep 18, 2025
@arnaudsm
Copy link
Owner

I'm merging your PR for now and will add the doc asap

@arnaudsm arnaudsm merged commit 969b478 into arnaudsm:main Sep 18, 2025
@brillout
Copy link
Contributor Author

Fix for root issue: remix-run/react-router#14351.

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