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
fix: serve and preview with base #8772
Conversation
✅ Deploy Preview for vite-docs-main ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
// redirect root visit to based url with search and hash | ||
res.writeHead(302, { | ||
Location: devBase + (parsed.search || '') + (parsed.hash || '') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
parsed.search
and parsed.hash
will never be null/undefined
.
This won't really work with SvelteKit. We have a Perhaps Vite should accept |
@benmccann It didn’t work at all previously so would it break SvelteKit? Of course I agree we should support base without slash. |
SvelteKit historically hasn't used the |
@sapphi-red Any progress on this? |
No, I've put this in the team board but not discussed yet. |
Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Closes vitejs/vite#9236 Closes vitejs/vite#8770 Closes vitejs/vite#8772
Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Closes vitejs/vite#9236 Closes vitejs/vite#8770 Closes vitejs/vite#8772
Description
close #8770
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).