Merge pull request #1917 from strapi/fix-build

Fix build

packages/strapi-admin/admin/src/containers/AdminPage/index.js

@@ -44,7 +44,7 @@ import ListPluginsPage from 'containers/ListPluginsPage/Loadable';
 import Logout from 'components/Logout';
 import NotFoundPage from 'containers/NotFoundPage/Loadable';
 import OverlayBlocker from 'components/OverlayBlocker';
-import PluginPage from 'containers/PluginPage/Loadable';
+import PluginPage from 'containers/PluginPage';
 
 // Utils
 import auth from 'utils/auth';

packages/strapi-admin/package.json

@@ -51,4 +51,4 @@
     "npm": ">= 5.0.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-api/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-controller/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-model/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-new/package.json

@@ -49,4 +49,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-plugin/package.json

@@ -44,4 +44,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-policy/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-generate-service/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-helper-plugin/lib/internals/webpack/webpack.dll.babel.js

@@ -69,6 +69,7 @@ module.exports = {
       'reactstrap': path.resolve(rootAdminpath, 'node_modules', 'strapi-helper-plugin', 'node_modules', 'reactstrap'),
       'react-dnd': path.resolve(rootAdminpath, 'node_modules', 'strapi-helper-plugin', 'node_modules', 'react-dnd'),
       'react-dnd-hmtl5-backend': path.resolve(rootAdminpath, 'node_modules', 'strapi-helper-plugin', 'node_modules', 'react-dnd-html5-backend'),
+      'styled-components': path.resolve(rootAdminpath, 'node_modules', 'strapi-helper-plugin', 'node_modules', 'styled-components'),
     },
     symlinks: false,
     extensions: [

packages/strapi-helper-plugin/lib/internals/webpack/webpack.prod.babel.js

@@ -238,11 +238,5 @@ module.exports = base({
 
   devtool: 'cheap-module-source-map',
   disableExtractTextPlugin: false,
-  externals: {
-    'styled-components': {
-      commonjs: 'styled-components',
-      commonjs2: 'styled-components',
-      amd: 'styled-components',
-    },
-  },
+  externals: {},
 });

packages/strapi-helper-plugin/lib/src/components/InputFile/index.js

@@ -137,7 +137,7 @@ class InputFile extends React.Component {
             position={this.state.position}
             updateFilePosition={this.updateFilePosition}
           />
-          <label style={{"margin-bottom": 0, width: '100%'}}>
+          <label style={{ marginBottom: 0, width: '100%' }}>
             <input
               className={styles.inputFile}
               multiple={multiple}

packages/strapi-helper-plugin/lib/src/components/InputFileWithErrors/index.js

@@ -44,10 +44,10 @@ class InputFileWithErrors extends React.PureComponent {
       this.updateState({ label: null });
     }
     // Check if errors have been updated during validations
-    if (nextProps.didCheckErrors !== this.props.didCheckErrors) {
+    if (prevProps.didCheckErrors !== this.props.didCheckErrors) {
       // Remove from the state the errors that have already been set
-      const errors = isEmpty(nextProps.errors) ? [] : nextProps.errors;
-      this.setState({ errors });
+      const errors = isEmpty(this.props.errors) ? [] : this.props.errors;
+      this.updateState({ errors });
     }
   }
 

packages/strapi-helper-plugin/lib/src/components/LoadingIndicatorPage/index.js

@@ -5,13 +5,28 @@
  */
 
 import React from 'react';
+import PropTypes from 'prop-types';
 
 import styles from './styles.scss';
 
-const LoadingIndicatorPage = () => {
+const LoadingIndicatorPage = (props) => {
+
+  if (props.error) {
+    console.log(props.error);
+    return <div>An error occurred</div>;
+  }
+
   return (
     <div className={styles.loaderPage}><div /></div>
   );
 };
 
+LoadingIndicatorPage.defaultProps = {
+  error: null,
+};
+
+LoadingIndicatorPage.propTypes = {
+  error: PropTypes.object,
+};
+
 export default LoadingIndicatorPage;

packages/strapi-helper-plugin/package.json

@@ -109,7 +109,7 @@
     "sass-loader": "^6.0.6",
     "shelljs": "^0.7.8",
     "style-loader": "^0.18.2",
-    "styled-components": "^3.2.6",
+    "styled-components": "3.2.6",
     "url-loader": "^0.5.9",
     "webpack": "^3.5.5",
     "webpack-bundle-analyzer": "^2.9.0",

packages/strapi-hook-bookshelf/package.json

@@ -55,4 +55,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-hook-ejs/package.json

@@ -43,4 +43,4 @@
     "npm": ">= 5.3.0"
   },
   "license": "MIT"
-}
+}

packages/strapi-plugin-content-manager/package.json

@@ -52,4 +52,4 @@
     "npm": ">= 5.0.0"
   },
   "license": "MIT"
-}
+}

packages/strapi/package.json

@@ -93,4 +93,4 @@
   },
   "preferGlobal": true,
   "license": "MIT"
-}
+}