fix(*): remove eslint disable lines

app/components/component-manuscript/src/components/ManuscriptPage.js

+import PropTypes from 'prop-types'
 import React from 'react'
 import { useQuery, gql } from '@apollo/client'
 import Manuscript from './Manuscript'
@@ -37,15 +38,17 @@ const query = gql`
   }
 `
 
-// eslint-disable-next-line react/prop-types
 const ManuscriptPage = ({ match, ...props }) => {
   const { data, loading, error } = useQuery(query, {
     variables: {
-      // eslint-disable-next-line react/prop-types
       id: match.params.version,
     },
   })
 
+  ManuscriptPage.propTypes = {
+    match: PropTypes.node.isRequired,
+  }
+
   if (loading) return <Spinner />
   if (error) return JSON.stringify(error)
   const { manuscript } = data

app/components/component-review/src/components/metadata/ReviewMetadata.js

-/* eslint-disable react/prop-types */
+import PropTypes from 'prop-types'
 import React from 'react'
 import styled from 'styled-components'
 import { get } from 'lodash'
@@ -129,4 +129,8 @@ const ReviewMetadata = ({ manuscript: rawManuscript }) => {
   )
 }
 
+ReviewMetadata.propTypes = {
+  manuscript: PropTypes.node.isRequired,
+}
+
 export default ReviewMetadata

app/components/component-submit/src/upload.js

 import config from 'config'
 import request from 'pubsweet-client/src/helpers/api'
-// eslint-disable-next-line import/no-extraneous-dependencies
 import gql from 'graphql-tag'
 import currentRolesVar from '../../../shared/currentRolesVar'
 
@@ -204,7 +203,6 @@ export default ({
   journals,
   currentUser,
   setConversion,
-  // eslint-disable-next-line consistent-return
 }) => async files => {
   setConversion({ converting: true })
   let manuscriptData
@@ -251,4 +249,6 @@ export default ({
   } catch (error) {
     setConversion({ error })
   }
+
+  return false
 }

app/components/xpub-journal/src/index.js

-/* eslint-disable react/destructuring-assignment */
+import PropTypes from 'prop-types'
 import React from 'react'
 
 const JournalContext = React.createContext()
 
-const JournalProvider = props => (
-  // eslint-disable-next-line react/prop-types
-  <JournalContext.Provider value={props.journal}>
-    {props.children}
-  </JournalContext.Provider>
+const JournalProvider = ({ journal, children }) => (
+  <JournalContext.Provider value={journal}>{children}</JournalContext.Provider>
 )
 
+JournalProvider.propTypes = {
+  journal : PropTypes.node.isRequired,
+  children : PropTypes.node.isRequired,
+}
+
 export { JournalContext, JournalProvider }