chore(react): proptypes

src/client/components/Button/index.jsx

@@ -12,7 +12,6 @@
  *  Copyright (c) 2014-2019. All rights reserved.
  */
 
-import isUndefined from 'lodash/isUndefined'
 import React from 'react'
 import PropTypes from 'prop-types'
 

src/client/components/Drowdown/DropdownSeperator.jsx

@@ -13,7 +13,6 @@
  */
 
 import React from 'react'
-import PropTypes from 'prop-types'
 
 class DropdownSeparator extends React.Component {
   render () {
@@ -25,6 +24,4 @@ class DropdownSeparator extends React.Component {
   }
 }
 
-DropdownSeparator.propTypes = {}
-
 export default DropdownSeparator

src/client/components/Drowdown/DropdownTrigger.jsx

@@ -46,7 +46,8 @@ class DropdownTrigger extends React.Component {
 
 DropdownTrigger.propTypes = {
   mode: PropTypes.string,
-  pos: PropTypes.string
+  pos: PropTypes.string,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 DropdownTrigger.defaultProps = {

src/client/components/Drowdown/index.jsx

@@ -29,7 +29,8 @@ class Dropdown extends React.Component {
 
 Dropdown.propTypes = {
   small: PropTypes.bool,
-  extraClass: PropTypes.string
+  extraClass: PropTypes.string,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default Dropdown

src/client/components/Grid/GridItem.jsx

@@ -29,7 +29,8 @@ class GridItem extends React.Component {
 GridItem.propTypes = {
   width: PropTypes.string.isRequired,
   extraClass: PropTypes.string,
-  style: PropTypes.object
+  style: PropTypes.object,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default GridItem

src/client/components/Grid/index.jsx

@@ -30,7 +30,8 @@ class Grid extends React.Component {
 
 Grid.propTypes = {
   extraClass: PropTypes.string,
-  style: PropTypes.object
+  style: PropTypes.object,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default Grid

src/client/components/Nav/Sidebar/index.jsx

@@ -82,7 +82,7 @@ class Sidebar extends React.Component {
   }
 
   render () {
-    const { activeItem, activeSubItem, plugins, sessionUser } = this.props
+    const { activeItem, activeSubItem, sessionUser } = this.props
     return (
       <ul className='side-nav'>
         {sessionUser && Helpers.canUser('agent:*', true) && (
@@ -297,7 +297,8 @@ Sidebar.propTypes = {
   updateNavChange: PropTypes.func.isRequired,
   activeItem: PropTypes.string.isRequired,
   activeSubItem: PropTypes.string.isRequired,
-  sessionUser: PropTypes.object
+  sessionUser: PropTypes.object,
+  plugins: PropTypes.array
 }
 
 const mapStateToProps = state => ({

src/client/components/Nav/SidebarItem/index.jsx

@@ -69,7 +69,8 @@ NavButton.propTypes = {
   class: PropTypes.string,
   hasSubmenu: PropTypes.bool,
   subMenuTarget: PropTypes.string,
-  active: PropTypes.bool
+  active: PropTypes.bool,
+  children: PropTypes.node.isRequired
 }
 
 export default NavButton

src/client/components/Nav/Submenu/index.jsx

@@ -96,9 +96,10 @@ class Submenu extends Component {
   }
 }
 
-Submenu.proptypes = {
+Submenu.propTypes = {
   id: PropTypes.string.isRequired,
-  subMenuOpen: PropTypes.bool
+  subMenuOpen: PropTypes.bool,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default Submenu

src/client/components/OffCanvas/OffCanvasTrigger.jsx

@@ -22,7 +22,8 @@ class OffCanvasTrigger extends React.Component {
 }
 
 OffCanvasTrigger.propTypes = {
-  target: PropTypes.string.isRequired
+  target: PropTypes.string.isRequired,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default OffCanvasTrigger

src/client/components/OffCanvas/index.jsx

@@ -35,7 +35,8 @@ class OffCanvas extends React.Component {
 
 OffCanvas.propTypes = {
   title: PropTypes.string,
-  id: PropTypes.string.isRequired
+  id: PropTypes.string.isRequired,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default OffCanvas

src/client/components/PDropdown/PDropdownTrigger.jsx

@@ -22,7 +22,8 @@ class PDropdownTrigger extends React.Component {
 }
 
 PDropdownTrigger.propTypes = {
-  target: PropTypes.string.isRequired
+  target: PropTypes.string.isRequired,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default PDropdownTrigger

src/client/components/PDropdown/index.jsx

@@ -58,7 +58,8 @@ PDropDown.propTypes = {
   topOffset: PropTypes.string,
   leftOffset: PropTypes.string,
   rightComponent: PropTypes.element,
-  footerComponent: PropTypes.element
+  footerComponent: PropTypes.element,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 PDropDown.defaultProps = {

src/client/components/Settings/SettingItem/index.jsx

@@ -79,7 +79,8 @@ SettingItem.propTypes = {
   tooltip: PropTypes.string,
   component: PropTypes.element,
   footer: PropTypes.element,
-  subPanelPadding: PropTypes.string
+  subPanelPadding: PropTypes.string,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default SettingItem

src/client/components/ZoneBox/index.jsx

@@ -23,7 +23,8 @@ class ZoneBox extends React.Component {
 }
 
 ZoneBox.propTypes = {
-  extraClass: PropTypes.string
+  extraClass: PropTypes.string,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default ZoneBox

src/client/components/ZoneBox/zone.jsx

@@ -23,7 +23,8 @@ class Zone extends React.Component {
 }
 
 Zone.propTypes = {
-  extraClass: PropTypes.string
+  extraClass: PropTypes.string,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default Zone

src/client/containers/Modals/BaseModal.jsx

@@ -69,7 +69,8 @@ BaseModal.propTypes = {
   options: PropTypes.object,
   modalTag: PropTypes.string,
   hideModal: PropTypes.func.isRequired,
-  clearModal: PropTypes.func.isRequired
+  clearModal: PropTypes.func.isRequired,
+  children: PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.node), PropTypes.node]).isRequired
 }
 
 export default connect(

src/client/containers/Topbar/onlineUserList.jsx

@@ -16,14 +16,13 @@ import React from 'react'
 import PropTypes from 'prop-types'
 import moment from 'moment-timezone'
 import { observer } from 'mobx-react'
-import { observable, computed, entries } from 'mobx'
-import { isUndefined, some } from 'lodash'
+import { observable, entries } from 'mobx'
+import { isUndefined } from 'lodash'
 
 import OffCanvas from 'components/OffCanvas'
 
 import UIkit from 'uikit'
 import socket from 'lib/socket'
-import helpers from 'lib/helpers'
 
 @observer
 class OnlineUserListPartial extends React.Component {