-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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!: put manifest files in .vite directory by default #14230
fix!: put manifest files in .vite directory by default #14230
Conversation
Run & review this pull request in StackBlitz Codeflow. |
/ecosystem-ci run |
📝 Ran ecosystem CI: Open
|
This just broke our build where the vite manifest was expected to be in the root folder, and we are still on 4.3.3. Weird... |
It looks like pnpm dlx isn't using the package.json file for vite, so somehow it is using version 5.0.0 instead of 4.3.3.... At least now I know where this is coming from... |
manifest.json default path got moved into a .vite folder cf vitejs/vite#14230
this pr introduces a bug in inertia-laravel in production. https://github.com/inertiajs/inertia-laravel Inertia expects manifest.json to be at public/build. a quick fix was to provide this path in vite.config |
@gcaraciolo you should report the issue to the repo. This is an intended breaking change in Vite 5. |
Description
Changes the default output path of manifest/ssrManifest to
.vite/manifest.json
/.vite/ssr-manifest.json
.fixes #9636
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).