#733 react-scripts: update to 3.0.1 (#734)

* #733 react-scripts: update to 3.0.1

* Removed deprecation notices associated history module.

* linter improvements, requires some proptypes to be locked down.

* eslint, requires fieldName to be corrected defined.

Author: martinfrances107

demo/docroot/sites/default/default.settings.php

@@ -754,6 +754,15 @@
  */
 $settings['entity_update_batch_size'] = 50;
 
+/**
+ * Entity update backup.
+ *
+ * This is used to inform the entity storage handler that the backup tables as
+ * well as the original entity type and field storage definitions should be
+ * retained after a successful entity update process.
+ */
+$settings['entity_update_backup'] = TRUE;
+
 /**
  * Load local development override configuration, if available.
  *

packages/admin-ui/package.json

@@ -27,7 +27,7 @@
     "deepmerge": "^3.0.0",
     "downshift": "^3.1.4",
     "emotion": "^10.0.0",
-    "history": "^4.7.2",
+    "history": "^4.9.0",
     "interweave": "^9.2.1",
     "jss": "^9.8.2",
     "keycode": "^2.2.0",
@@ -65,7 +65,7 @@
     "chromedriver": "^2.45.0",
     "enzyme": "^3.3.0",
     "enzyme-adapter-react-16": "^1.1.1",
-    "eslint": "5.6.0",
+    "eslint": "5.16.0",
     "eslint-config-airbnb": "17.1.0",
     "eslint-config-prettier": "^3.1.0",
     "eslint-plugin-import": "^2.7.0",
@@ -76,7 +76,7 @@
     "nightwatch": "^1.0.16",
     "prettier": "^1.11.1",
     "react-axe": "^3.0.1",
-    "react-scripts": "2.1.1",
+    "react-scripts": "3.0.1",
     "react-test-renderer": "^16.6.3",
     "redux-saga-test-plan": "^3.6.0",
     "storybook": "^1.0.0",

packages/admin-ui/src/App.js

@@ -1,7 +1,8 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { Route, Switch, withRouter } from 'react-router-dom';
-import createBrowserHistory from 'history/createBrowserHistory';
+import { createBrowserHistory } from 'history';
+
 import deepMerge from 'deepmerge';
 import { createStore, applyMiddleware, combineReducers } from 'redux';
 import {

packages/components/src/Components/FormElements/InputCheckbox/InputCheckbox.js

@@ -142,25 +142,33 @@ const InputCheckbox = props => {
 };
 
 InputCheckbox.propTypes = {
+  checked: PropTypes.bool,
+
   children: PropTypes.oneOfType([
     PropTypes.arrayOf(PropTypes.node),
     PropTypes.node,
   ]).isRequired,
 
   error: PropTypes.bool,
 
+  fieldName: PropTypes.string.isRequired,
+
   htmlAttributes: PropTypes.objectOf([
     PropTypes.string,
     PropTypes.number,
     PropTypes.bool,
   ]),
 
+  onChange: PropTypes.func,
+
   required: PropTypes.bool,
 };
 
 InputCheckbox.defaultProps = {
+  checked: false,
   error: false,
   htmlAttributes: {},
+  onChange: () => {},
   required: false,
 };
 

packages/components/src/Components/FormElements/InputRadio/InputRadio.js

@@ -156,6 +156,8 @@ const InputRadio = props => {
 };
 
 InputRadio.propTypes = {
+  checked: PropTypes.bool,
+
   /** Content contained within the label element. */
   children: PropTypes.oneOfType([
     PropTypes.arrayOf(PropTypes.node),
@@ -165,6 +167,8 @@ InputRadio.propTypes = {
   /** true if the label relates to an element with an error. */
   error: PropTypes.bool,
 
+  fieldName: PropTypes.string.isRequired,
+
   /** Any additional HTML properties to add to the label element. */
   htmlAttributes: PropTypes.objectOf([
     PropTypes.string,
@@ -175,13 +179,17 @@ InputRadio.propTypes = {
   /** Content of the html "for" attribute to use for this label. */
   htmlFor: PropTypes.string.isRequired,
 
+  onChange: PropTypes.func,
+
   /** True if this label relates to an element that is required. */
   required: PropTypes.bool,
 };
 
 InputRadio.defaultProps = {
+  checked: false,
   error: false,
   htmlAttributes: {},
+  onChange: () => {},
   required: false,
 };