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

Basic Routing Works on Server but not Client #9276

Closed
jdgamble555 opened this issue Apr 21, 2024 · 4 comments
Closed

Basic Routing Works on Server but not Client #9276

jdgamble555 opened this issue Apr 21, 2024 · 4 comments

Comments

@jdgamble555
Copy link

jdgamble555 commented Apr 21, 2024

Reproduction

https://remix-firebase-test.vercel.app/about

https://github.com/jdgamble555/remix-firebase-test/tree/master

https://stackblitz.com/github/jdgamble555/remix-firebase-test

The stackblitz seems to work fine. The problem is ONLY in production. You can test this on a local machine by running:

  1. npm run build
  2. npm run start
  3. Navigate here: http://localhost:3000/about

System Info

Pushed to Vercel, Brand new remix - 2.8.1

Used Package Manager

npm

This is a basic app. All I have installed is Tailwind.

Expected Behavior

It should just display the route as working in the dev version.

Actual Behavior

It says 404 not found, the server loads correctly, but the browser routing is not correct.

@huijiewei
Copy link

override rollup to 4.15.0, this is rollup issue.

@jdgamble555
Copy link
Author

Since rollup is in the core package though, Remix should upgrade their dependencies. But yes, this fixed the problem.

@brophdawg11
Copy link
Contributor

rollup is not in the core package, it's a nested dependency of vite. The rollup folks are looking into this and have a fix open: rollup/rollup#5443 (comment).

Closing as a duplicate of #9275

@brophdawg11 brophdawg11 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 22, 2024
@brophdawg11
Copy link
Contributor

Resolved in rollup 4.16.2: https://github.com/rollup/rollup/releases/tag/v4.16.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants