-
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
SPA mode + basename feature gives match issue in dev and 404 error on build #8712
Comments
Ah ok yeah I bet this is a one line change to add the https://github.com/remix-run/remix/blob/dev/packages/remix-dev/vite/plugin.ts#L1718 |
This does the trick @brophdawg11 Added the patch to here: https://stackblitz.com/edit/remix-run-remix-bqalcj but for some reason it won't pick it up for the dev build on Stackblitz, but works fine on the prod build. Both work fine locally. |
Fixed in #8720 |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
Reproduction
Related to #8145
Dev build issue
Production build issue
npm run build
and see:System Info
Used Package Manager
npm
Expected Behavior
Actual Behavior
The text was updated successfully, but these errors were encountered: