-
Notifications
You must be signed in to change notification settings - Fork 592
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(nms): Make sure hot module replacement works for the e2e test #13572
fix(nms): Make sure hot module replacement works for the e2e test #13572
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
33263a7
to
8a8a092
Compare
nms/babel.config.js
Outdated
@@ -12,7 +12,7 @@ | |||
*/ | |||
|
|||
function isWebpack(caller) { | |||
return !!(caller && caller.name === 'web'); |
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.
caller.target
was web
so the plugin was never enabled. Somehow it work without, but I kept it because the docs say you should use it.
}; | ||
|
||
void fetchNmsVersion(); | ||
}); |
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.
prevent unnecessary requests
magma#13559)" Signed-off-by: Thomas Schmitt <thomas.schmitt@tngtech.com> This reverts commit b2680c8.
Signed-off-by: Thomas Schmitt <thomas.schmitt@tngtech.com>
8a8a092
to
ea2377f
Compare
Signed-off-by: Thomas Schmitt <thomas.schmitt@tngtech.com>
Summary
This re-enables the hot module replacement as it was initially configured here #13559
The e2e tests were broken because there was an unhandled exception that lead to an overlay that blocked the UI. This has been fixed.
Test Plan