Merge pull request #3019 from thornbill/enable-multiserver-dev

Enable multiserver in development environments

src/scripts/settings/webSettings.js

@@ -71,6 +71,11 @@ export function getIncludeCorsCredentials() {
 }
 
 export function getMultiServer() {
+    // Enable multi-server support when served by webpack
+    if (__WEBPACK_SERVE__) { // eslint-disable-line no-undef
+        return Promise.resolve(true);
+    }
+
     return getConfig().then(config => {
         return !!config.multiserver;
     }).catch(error => {

src/serviceworker.js

@@ -44,9 +44,8 @@ self.addEventListener('notificationclick', function (event) {
     event.waitUntil(executeAction(action, data, serverId));
 }, false);
 
-// Do not precache files in development so live reload works as expected
+// Do not precache files when running with webpack dev server so live reload works as expected
-/* eslint-disable-next-line no-undef -- NODE_ENV is replaced by webpack */
+if (!__WEBPACK_SERVE__) { // eslint-disable-line no-undef
-if (process.env.NODE_ENV === 'production') {
     // this is needed by the webpack Workbox plugin
     /* eslint-disable-next-line no-restricted-globals,no-undef */
     precacheAndRoute(self.__WB_MANIFEST);

webpack.common.js

@@ -2,6 +2,7 @@ const path = require('path');
 const { CleanWebpackPlugin } = require('clean-webpack-plugin');
 const CopyPlugin = require('copy-webpack-plugin');
 const HtmlWebpackPlugin = require('html-webpack-plugin');
+const { DefinePlugin } = require('webpack');
 
 const Assets = [
     'native-promise-only/npo.js',
@@ -30,6 +31,9 @@ module.exports = {
         ]
     },
     plugins: [
+        new DefinePlugin({
+            __WEBPACK_SERVE__: JSON.stringify(!!process.env.WEBPACK_SERVE)
+        }),
         new CleanWebpackPlugin(),
         new HtmlWebpackPlugin({
             filename: 'index.html',