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

Fix hmr server port #21344

Closed
wants to merge 4 commits into from
Closed

Fix hmr server port #21344

wants to merge 4 commits into from

Conversation

ghostdevv
Copy link

@ghostdevv ghostdevv commented Jun 4, 2023

This fixes #21341 by removing some code that set vite's hmr port to 24678 everytime. In Vite 3 this pr set the default behaviour to use the clientPort (defaults to 3000 in nuxt) for HMR rather than a 24678. Removing the code in nuxt's vite builder that set this port makes vite's default behaviour kick in causing #21341 to be fixed

@ghostdevv ghostdevv marked this pull request as draft June 4, 2023 00:07
@ghostdevv ghostdevv changed the title fix: remove old hmr config Fix hmr server port Jun 4, 2023
@ghostdevv
Copy link
Author

ghostdevv commented Jun 4, 2023

Trying to workout why I sometimes see this error, it doesn't appear to happen all the time
image

@ghostdevv
Copy link
Author

HMR in the playground also seems to not work at all, even when I try locally on main 🤔

@danielroe danielroe added the 3.x label Jun 4, 2023
@ghostdevv ghostdevv closed this Oct 7, 2023
@ghostdevv ghostdevv deleted the fix-hmr branch October 7, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HMR in Gitpod + VS Code Browser
2 participants