MM-63616 Remove react-hot-loader and hot-loader/react-dom (#30744)

webapp/channels/.npm-upgrade.json

@@ -80,10 +80,6 @@
       "versions": "",
       "reason": "MM-34914 - Breaking changes to how modules are loaded."
     },
-    "@hot-loader/react-dom": {
-      "versions": "",
-      "reason": "MM-45255 - React updates are handled separately from other dependencies."
-    },
     "@types/marked": {
       "versions": "",
       "reason": "Ignored since our forked version is wildly different from upstream."
@@ -105,4 +101,4 @@
       "reason": "MM-37116 - Breaking changes in API"
     }
   }
-}
+}

webapp/channels/babel.config.js

@@ -23,7 +23,6 @@ const config = {
         }],
     ],
     plugins: [
-        'react-hot-loader/babel',
         'babel-plugin-typescript-to-proptypes',
         [
             'babel-plugin-styled-components',

webapp/channels/package.json

@@ -66,7 +66,6 @@
     "react-custom-scrollbars": "4.2.1",
     "react-day-picker": "8.3.6",
     "react-dom": "17.0.2",
-    "react-hot-loader": "4.13.0",
     "react-intl": "*",
     "react-is": "17.0.2",
     "react-overlays": "0.9.3",
@@ -98,7 +97,6 @@
   },
   "devDependencies": {
     "@deanwhillier/jest-matchmedia-mock": "1.2.0",
-    "@hot-loader/react-dom": "17.0.2",
     "@mattermost/calls-common": "0.27.0",
     "@mattermost/eslint-plugin": "*",
     "@mattermost/mmjstool": "github:mattermost/mattermost-utilities#7b63833d208d482ba4a1c12230bb3e68dd9c5e5e",

webapp/channels/src/components/app.tsx

@@ -2,7 +2,6 @@
 // See LICENSE.txt for license information.
 
 import React, {lazy} from 'react';
-import {hot} from 'react-hot-loader/root';
 import {Provider} from 'react-redux';
 import {Router} from 'react-router-dom';
 
@@ -25,4 +24,4 @@ const App = () => {
     );
 };
 
-export default hot(React.memo(App));
+export default React.memo(App);

webapp/channels/webpack.config.js

@@ -274,7 +274,7 @@ function generateCSP() {
     let csp = 'script-src \'self\' cdn.rudderlabs.com/ js.stripe.com/v3';
 
     if (DEV) {
-        // react-hot-loader and development source maps require eval
+        // Development source maps require eval
         csp += ' \'unsafe-eval\'';
     }
 
@@ -429,13 +429,6 @@ if (targetIsDevServer) {
             ...config.optimization,
             splitChunks: false,
         },
-        resolve: {
-            ...config.resolve,
-            alias: {
-                ...config.resolve.alias,
-                'react-dom': '@hot-loader/react-dom',
-            },
-        },
     };
 }