if adding an attachment to a request upon creation results in an error, pass that error to the request show page and handle it

Author: alishaevn

pages/requests/[uuid].js

@@ -33,7 +33,7 @@ const Request = ({ session }) => {
    * as a dynamically routed file, the router query will always consist of a "key: value" pair that's determined by the name of
    * the file (key) and path string (value). additional query properties may also exist if they were explicitly passed.
   */
-  const { uuid } = router.query
+  const { createRequestError, uuid } = router.query
   const accessToken = session?.accessToken
   const { request, isLoadingRequest, isRequestError } = useOneRequest(uuid, accessToken)
   const { allSOWs, isLoadingSOWs, isSOWError } = useAllSOWs(uuid, request?.identifier, accessToken)
@@ -78,6 +78,33 @@ const Request = ({ session }) => {
     )
   }
 
+  // this error is a result of creating the request
+  if (createRequestError) {
+    // TODO(alishaevn): how to handle a "sent to vendor" error?
+    const attachmentError = JSON.parse(createRequestError).config.url.includes('notes')
+
+    return (
+      <Notice
+        alert={attachmentError &&
+          {
+            body: ['Your attachment failed. Please visit your request and try again from the "View Files" tab.'],
+            title: 'Attachment Error',
+            variant: 'info',
+          }
+        }
+        dismissible={false}
+        withBackButton={true}
+        buttonProps={attachmentError &&
+          {
+            onClick: () => router.push({ pathname: `/requests/${uuid}` }),
+            text: 'Click to view your request.',
+          }
+        }
+      />
+    )
+  }
+
+  // this error is a result of getting the request or its related data
   if (isError) {
     return (
       <Notice

pages/requests/new/[ware].js

@@ -99,22 +99,34 @@ const NewRequest = ({ session }) => {
       wareID,
       accessToken,
     })
-    if (error) return setCreateRequestError(error)
+    // if we have data AND an error, the request was created, but the attachments failed
+    // in that case, we still need to send the request to the vendor
+    if (error && !data) {
+      setFormSubmitting(false)
+      setCreateRequestError(error)
+      return
+    } else if (error) {
+      setCreateRequestError(error)
+    }
 
     const sentToVendor = await sendRequestToVendor(data.id, accessToken)
-    if (sentToVendor.error) return setCreateRequestError(sentToVendor.error)
+    if (sentToVendor.error) {
+      setFormSubmitting(false)
+      setCreateRequestError(sentToVendor.error)
+    }
 
     setCreatedRequestUUID(data.uuid)
   }
 
   useEffect(() => {
     if (createdRequestUUID) {
       router.push({
-        pathname: `/requests/${createdRequestUUID}`
-      })
+        pathname: `/requests/${createdRequestUUID}`,
+        query: { createRequestError: JSON.stringify(createRequestError) },
+        }, `/requests/${createdRequestUUID}`)
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [createdRequestUUID])
+  }, [createdRequestUUID, createRequestError])
 
   // TODO(alishaevn): use react bs placeholder component
   if (isLoadingInitialRequest || !wareID || formSubmitting) return <Loading wrapperClass='item-page mt-5' />