feat(publonsInvite): refactored to use error

packages/component-faraday-ui/src/PublonsTable.js

@@ -7,8 +7,20 @@ import { compose, withHandlers, withProps } from 'recompose'
 
 import { Label, OpenModal, Text, withFetching } from '../'
 
-const TableView = ({ reviewers, onInviteReviewer, setFetching, isFetching }) =>
-  reviewers.length === 0 ? (
+const TableView = ({
+  reviewers,
+  onInviteReviewer,
+  setFetching,
+  isFetching,
+  publonsError,
+}) => {
+  if (publonsError)
+    return (
+      <Text align="center" error>
+        {publonsError}
+      </Text>
+    )
+  return reviewers.length === 0 ? (
     <Text align="center">No suggestions yet.</Text>
   ) : (
     <Table>
@@ -58,6 +70,7 @@ const TableView = ({ reviewers, onInviteReviewer, setFetching, isFetching }) =>
       </tbody>
     </Table>
   )
+}
 
 const PublonsTable = ({ publonsFetching, ...rest }) => (
   <Fragment>{publonsFetching ? <Spinner /> : <TableView {...rest} />}</Fragment>

packages/component-faraday-ui/src/contextualBoxes/ReviewerDetails.js

@@ -29,6 +29,7 @@ const ReviewerDetails = ({
   isFetching,
   previewFile,
   downloadFile,
+  fetchingError,
   canInviteReviewers,
   onInviteReviewer,
   onInvitePublonReviewer,
@@ -93,6 +94,7 @@ const ReviewerDetails = ({
               {selectedTab === 1 && (
                 <PublonsTable
                   onInvite={onInvitePublonReviewer}
+                  publonsError={fetchingError}
                   publonsFetching={isFetching}
                   reviewers={publonReviewers}
                 />

packages/component-manuscript/src/components/ManuscriptLayout.js

@@ -44,6 +44,7 @@ const ManuscriptLayout = ({
   fragment = {},
   changeForm,
   isFetching,
+  fetchingError,
   formValues,
   heExpanded,
   onHEResponse,
@@ -175,6 +176,7 @@ const ManuscriptLayout = ({
         {get(currentUser, 'permissions.canViewReviewersDetails', false) && (
           <ReviewerDetails
             currentUser={currentUser}
+            fetchingError={fetchingError}
             fragment={fragment}
             getSignedUrl={getSignedUrl}
             invitations={invitationsWithReviewers}

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

@@ -65,7 +65,7 @@ import {
   parseEicDecision,
   parseSearchParams,
   redirectToError,
-  handlePublons,
+  getPublonsReviewers,
 } from './utils'
 import {
   canAssignHE,
@@ -395,6 +395,8 @@ export default compose(
       fetchUpdatedCollection,
       setPublons,
       setFetching: setListFetching,
+      setError,
+      clearError,
     }) => (values, { hideModal, setModalError, setFetching }) => {
       setFetching(true)
       inviteReviewer({
@@ -407,10 +409,12 @@ export default compose(
           setFetching(false)
           hideModal()
           fetchUpdatedCollection()
-          handlePublons({
+          getPublonsReviewers({
             fragmentId: fragment.id,
             setPublons,
             setFetching: setListFetching,
+            setError,
+            clearError,
           })
         })
         .catch(err => {
@@ -459,6 +463,8 @@ export default compose(
           permissions: { canInviteReviewers },
         },
         setFetching,
+        setError,
+        clearError,
       } = this.props
 
       if (hasManuscriptFailure) {
@@ -481,7 +487,13 @@ export default compose(
       )
 
       if (canInviteReviewers) {
-        handlePublons({ fragmentId, setPublons, setFetching })
+        getPublonsReviewers({
+          fragmentId,
+          setPublons,
+          setFetching,
+          setError,
+          clearError,
+        })
       }
 
       if (isInvitedHE) {

packages/component-manuscript/src/components/utils.js

@@ -306,13 +306,22 @@ export const parseEicDecision = ({ decision, message }) => ({
 })
 
 // handle publons
-export const getPublons = fragmentId =>
+export const getPublonsAPI = fragmentId =>
   apiGet(`/fragments/${fragmentId}/publons`)
 
-export const handlePublons = ({ fragmentId, setPublons, setFetching }) => {
+export const getPublonsReviewers = ({
+  fragmentId,
+  setPublons,
+  setFetching,
+  setError,
+  clearError,
+}) => {
+  clearError()
   setFetching(true)
-  getPublons(fragmentId).then(res => {
-    setPublons(res)
-    setFetching(false)
-  })
+  getPublonsAPI(fragmentId)
+    .then(res => {
+      setPublons(res)
+      setFetching(false)
+    })
+    .catch(handleError(setError))
 }