fix: move react-dom notification to AppContainer, fixes #1227

Author: theKashey

Diff for: src/AppContainer.dev.js

@@ -28,6 +28,14 @@ class AppContainer extends React.Component {
     generation: 0,
   };
 
+  constructor(props) {
+    super(props);
+    if (configuration.showReactDomPatchNotification) {
+      configuration.showReactDomPatchNotification = false;
+      console.warn('React-Hot-Loader: react-🔥-dom patch is not detected. React 16.6+ features may not work.');
+    }
+  }
+
   shouldComponentUpdate(prevProps, prevState) {
     // Don't update the component if the state had an error and still has one.
     // This allows to break an infinite loop of error -> render -> error -> render

Diff for: src/configuration.js

@@ -17,6 +17,9 @@ const configuration = {
   // Disable "hot-replacement-render" when injection into react-dom is made
   disableHotRendererWhenInjected: false,
 
+  // Controls `react-🔥-dom patch` notification
+  showReactDomPatchNotification: true,
+
   // Hook on babel component register.
   onComponentRegister: false,
 

Diff for: src/reactHotLoader.js

@@ -106,10 +106,8 @@ const reactHotLoader = {
       configuration.ignoreSFC = configuration.ignoreSFCWhenInjected;
 
       reactHotLoader.IS_REACT_MERGE_ENABLED = true;
+      configuration.showReactDomPatchNotification = false;
       // console.warn('react-🔥-loader activated.');
-    } else {
-      // Actually everything works...
-      console.warn('React-Hot-Loader: react-🔥-dom patch is not detected. React 16.6+ features may not work.');
     }
     /* eslint-enable */
     if (!React.createElement.isPatchedByReactHotLoader) {