Merge pull request #1692 from KKVANONYMOUS/develop

Fix PropTypes.object() linting errors

Author: catarak

Diff for: client/components/mobile/ActionStrip.jsx

@@ -38,7 +38,7 @@ const ActionStrip = ({ actions }) => (
 
 ActionStrip.propTypes = {
   actions: PropTypes.arrayOf(PropTypes.shape({
-    icon: PropTypes.any,
+    icon: PropTypes.component,
     aria: PropTypes.string.isRequired,
     action: PropTypes.func.isRequired,
     inverted: PropTypes.bool

Diff for: client/modules/IDE/components/NewFileForm.jsx

@@ -58,7 +58,7 @@ class NewFileForm extends React.Component {
 
 NewFileForm.propTypes = {
   fields: PropTypes.shape({
-    name: PropTypes.object.isRequired
+    name: PropTypes.objectOf(PropTypes.shape()).isRequired
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   createFile: PropTypes.func.isRequired,

Diff for: client/modules/IDE/components/NewFolderForm.jsx

@@ -56,7 +56,7 @@ class NewFolderForm extends React.Component {
 
 NewFolderForm.propTypes = {
   fields: PropTypes.shape({
-    name: PropTypes.object.isRequired
+    name: PropTypes.objectOf(PropTypes.shape()).isRequired
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   createFolder: PropTypes.func.isRequired,

Diff for: client/modules/IDE/pages/IDEView.jsx

@@ -495,7 +495,7 @@ IDEView.propTypes = {
     updatedAt: PropTypes.string,
   }).isRequired,
   editorAccessibility: PropTypes.shape({
-    lintMessages: PropTypes.array.isRequired, // eslint-disable-line
+    lintMessages: PropTypes.objectOf(PropTypes.shape()).isRequired,
   }).isRequired,
   preferences: PropTypes.shape({
     autosave: PropTypes.bool.isRequired,

Diff for: client/modules/User/components/APIKeyForm.jsx

@@ -7,8 +7,8 @@ import CopyableInput from '../../IDE/components/CopyableInput';
 import APIKeyList from './APIKeyList';
 
 export const APIKeyPropType = PropTypes.shape({
-  id: PropTypes.object.isRequired, // eslint-disable-line
-  token: PropTypes.object, // eslint-disable-line
+  id: PropTypes.objectOf(PropTypes.shape()).isRequired,
+  token: PropTypes.objectOf(PropTypes.shape()),
   label: PropTypes.string.isRequired,
   createdAt: PropTypes.string.isRequired,
   lastUsedAt: PropTypes.string

Diff for: client/modules/User/components/AccountForm.jsx

@@ -109,10 +109,10 @@ function AccountForm(props) {
 
 AccountForm.propTypes = {
   fields: PropTypes.shape({
-    username: PropTypes.object.isRequired, // eslint-disable-line
-    email: PropTypes.object.isRequired, // eslint-disable-line
-    currentPassword: PropTypes.object.isRequired, // eslint-disable-line
-    newPassword: PropTypes.object.isRequired, // eslint-disable-line
+    username: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    email: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    currentPassword: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    newPassword: PropTypes.objectOf(PropTypes.shape()).isRequired,
   }).isRequired,
   user: PropTypes.shape({
     verified: PropTypes.string.isRequired,

Diff for: client/modules/User/components/LoginForm.jsx

@@ -54,8 +54,8 @@ function LoginForm(props) {
 
 LoginForm.propTypes = {
   fields: PropTypes.shape({
-    email: PropTypes.object.isRequired, // eslint-disable-line
-    password: PropTypes.object.isRequired, // eslint-disable-line
+    email: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    password: PropTypes.objectOf(PropTypes.shape()).isRequired,
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   validateAndLoginUser: PropTypes.func.isRequired,

Diff for: client/modules/User/components/NewPasswordForm.jsx

@@ -47,8 +47,8 @@ function NewPasswordForm(props) {
 
 NewPasswordForm.propTypes = {
   fields: PropTypes.shape({
-    password: PropTypes.object.isRequired, // eslint-disable-line
-    confirmPassword: PropTypes.object.isRequired, // eslint-disable-line
+    password: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    confirmPassword: PropTypes.objectOf(PropTypes.shape()).isRequired,
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   updatePassword: PropTypes.func.isRequired,

Diff for: client/modules/User/components/ResetPasswordForm.jsx

@@ -37,7 +37,7 @@ function ResetPasswordForm(props) {
 
 ResetPasswordForm.propTypes = {
   fields: PropTypes.shape({
-    email: PropTypes.object.isRequired
+    email: PropTypes.objectOf(PropTypes.shape()).isRequired
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   initiateResetPassword: PropTypes.func.isRequired,

Diff for: client/modules/User/components/SignupForm.jsx

@@ -83,10 +83,10 @@ function SignupForm(props) {
 
 SignupForm.propTypes = {
   fields: PropTypes.shape({
-    username: PropTypes.object.isRequired, // eslint-disable-line
-    email: PropTypes.object.isRequired, // eslint-disable-line
-    password: PropTypes.object.isRequired, // eslint-disable-line
-    confirmPassword: PropTypes.object.isRequired, // eslint-disable-line
+    username: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    email: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    password: PropTypes.objectOf(PropTypes.shape()).isRequired,
+    confirmPassword: PropTypes.objectOf(PropTypes.shape()).isRequired,
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,
   signUpUser: PropTypes.func.isRequired,