Repository navigation
[OGUI-1939] Add Vite dev server and HMR integration - #3701
Open
isaachilly wants to merge 1 commit into
Open
isaachilly wants to merge 1 commit into
isaachilly wants to merge 1 commit into
Conversation
isaachilly
force-pushed
the
feature/ILG/OGUI-1939/vite-dev-server
branch
from
November 10, 2026 15:45
8add018 to
74ca3e5
Compare
isaachilly
force-pushed
the
feature/ILG/OGUI-1939/vite-dev-server
branch
from
November 10, 2026 15:59
74ca3e5 to
cddc326
Compare
isaachilly
marked this pull request as ready for review
November 11, 2026 07:51
|
|
||
| // Dev only: swap in the new view tree on edit and keep model state; stripped from the prod build | ||
| if (import.meta.hot) { | ||
| import.meta.hot.accept('./view.js', (newModule) => { |
isaachilly
force-pushed
the
feature/ILG/OGUI-1939/vite-dev-server
branch
from
November 11, 2026 18:58
cddc326 to
996a43c
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I have JIRA issue created
Ticket:
Integrate the Vite development server into ILG and fix any compatibility issues. Frontend changes can then show up in the browser straight away with no rebuild or reload.
Code:
When
NODE_ENV= development, the frontend is served not as the Vitedist/but through Vite middleware with the HMR WebSocket live as well./api/*skip the Vite middlewareRoot view is mounted through a reference so that Vite's HMR can swap in a new view tree and keep old model state.
Use node watch instead of nodemon.
Reviewer:
Requires Framework release as it depends on the
extraConnectSrcfor Vite's HMR to be allowed by our CSP.