-
Notifications
You must be signed in to change notification settings - Fork 164
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
Live reload on frontend changes #8875
Conversation
8eab654
to
bb3ced6
Compare
When the webpack dev server finished recompilation, it notifies the Java server over the watchdog connection. When the server notices the message, it sends a browser reload message over the dedicated live reload web socket connection. Other changes in this commit: - live reload support for V14 bootstrapping mode - vaadin-devmode-gizmo is now bundled separately
bb3ced6
to
91209f1
Compare
@@ -72,18 +73,21 @@ | |||
* name of the JS file to update with the Flow project imports | |||
* @param useV14Bootstrapping | |||
* whether the application running with deprecated V14 bootstrapping | |||
* @param flowResourcesFolder | |||
* relative path to `flow-frontend` package | |||
*/ | |||
TaskUpdateWebpack(File frontendDirectory, File webpackConfigFolder, |
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.
@@ -79,6 +84,19 @@ private Logger getLogger() { | |||
return LoggerFactory.getLogger(WatchDogServer.class); | |||
} | |||
|
|||
private void enterReloadMessageReadLoop(Socket accept) throws IOException{ | |||
BufferedReader in = new BufferedReader(new InputStreamReader(accept.getInputStream())); |
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.
@@ -406,6 +406,12 @@ | |||
@Deprecated | |||
public static final String SERVLET_PARAMETER_DEVMODE_ENABLE_LIVE_RELOAD = InitParameters.SERVLET_PARAMETER_DEVMODE_ENABLE_LIVE_RELOAD; |
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.
flow-client/src/main/java/com/vaadin/client/ApplicationConfiguration.java
Show resolved
Hide resolved
SonarQube analysis reported 34 issues Watch the comments in this conversation to review them. Top 10 extra issuesNote: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:
|
Trigger live reload when webpack recompiles frontend files When the webpack dev server finished recompilation, it notifies the Java server over the watchdog connection. When the server notices the message, it sends a browser reload message over the dedicated live reload web socket connection. Other changes in this commit: - live reload support for V14 bootstrapping mode - vaadin-devmode-gizmo is now bundled separately - Convert VaadinDevmodeGizmo to TypeScript
Trigger live reload when webpack recompiles frontend files When the webpack dev server finished recompilation, it notifies the Java server over the watchdog connection. When the server notices the message, it sends a browser reload message over the dedicated live reload web socket connection. Other changes in this commit: - live reload support for V14 bootstrapping mode - vaadin-devmode-gizmo is now bundled separately - Convert VaadinDevmodeGizmo to TypeScript
When/if backporting to 2.3, check compatibility with ES5 transpilation (e.g. |
When the webpack dev server finished recompilation, it will notify the Java server over
the watchdog connection. When the Java server notices the message, it in turn sends
the browser reload message over the dedicated live reload web socket connections.
Other changes in this commit:
Addresses tickets #8852, #8721, #8001, #5617, #607 (and maybe others).