Merge pull request #254 from scientist-softserv/138-diplay-pos

138 diplay POs

Author: summer-cook

pages/api/auth/[...nextauth].js

@@ -65,8 +65,10 @@ const refreshAccessToken = async (token) => {
     const url = `https://${process.env.NEXT_PUBLIC_PROVIDER_NAME}.scientist.com/oauth/token`
     const encodedString = Buffer.from(`${process.env.CLIENT_ID}:${process.env.CLIENT_SECRET}`).toString('base64')
     const params = new URLSearchParams({
+      /* eslint-disable camelcase */
       grant_type: 'refresh_token',
       refresh_token: token.refreshToken,
+      /* eslint-enable camelcase */
     })
 
     const response = await axios.post(url, params, {

pages/requests/[uuid].js

@@ -1,4 +1,4 @@
-import React from 'react'
+import React, { useState, useEffect } from 'react'
 import { useRouter } from 'next/router'
 import { useSession } from 'next-auth/react'
 import {
@@ -22,6 +22,7 @@ import {
   statusBarBg,
   useMessages,
   useFiles,
+  getAllPOs,
   useAllSOWs,
   useOneRequest,
 } from '../../utils'
@@ -38,10 +39,25 @@ const Request = () => {
   const { allSOWs, isLoadingSOWs, isSOWError } = useAllSOWs(uuid, request?.identifier, session?.accessToken)
   const { messages, isLoadingMessages, isMessagesError, mutateMessages, messagesData } = useMessages(uuid, session?.accessToken)
   const { files, isLoadingFiles, isFilesError, mutateFiles, filesData } = useFiles(uuid, session?.accessToken)
-  const documents = (allSOWs) ? [...allSOWs] : []
 
-  const isLoading = isLoadingRequest || isLoadingSOWs || isLoadingFiles || isLoadingMessages
-  const isError = isRequestError || isSOWError || isFilesError|| isMessagesError
+  const [allPOs, setAllPOs] = useState([])
+  const [isPOError, setIsPOError] = useState(false)
+  const [isLoadingPOs, setIsLoadingPOs] = useState(true)
+  useEffect(() => {
+    if (request) {
+      (async () => {
+        const { allPOs, isLoadingPOs, isPOError } = await getAllPOs(request?.quotedWareID, uuid, request?.identifier, session?.accessToken)
+
+        setIsLoadingPOs(isLoadingPOs)
+        setAllPOs(allPOs)
+        setIsPOError(isPOError)
+      })()
+    }
+  }, [allPOs, isPOError, request, uuid, session])
+
+  const isLoading = isLoadingRequest || isLoadingSOWs || isLoadingFiles || isLoadingMessages || isLoadingPOs
+  const isError = isRequestError || isSOWError || isFilesError|| isMessagesError || isPOError
+  const documents = (allSOWs) ? [...allSOWs, ...allPOs] : []
 
   if (isLoading) return <Loading wrapperClass='item-page mt-5' />
 
@@ -61,7 +77,7 @@ const Request = () => {
   if (isError) {
     return (
       <Notice
-        alert={configureErrors([isRequestError, isSOWError, isMessagesError, isFilesError])}
+        alert={configureErrors([isRequestError, isSOWError, isMessagesError, isFilesError, isPOError])}
         dismissible={false}
         withBackButton={true}
         buttonProps={{

utils/api/configurations.js

@@ -185,20 +185,16 @@ export const  configureFiles = (data) => {
   return allFiles
 }
 
-export const configureDocuments = (documents, requestIdentifier) => {
-  return documents?.map(document => ({
-    identifier: document.identifier,
+export const configureDocument = (document, requestIdentifier) => {
+  return {
     date: normalizeDate(document.created_at),
     documentStatus: document.status,
     documentStatusColor: statusColors[configureStatus(document.status)].bg,
     documentType: document.type,
     documentTypeColor: 'bg-dark',
+    identifier: document.identifier,
     lineItems: configureLineItems(document.line_items),
     requestIdentifier,
-    subtotalPrice: document.retail_subtotal_price_currency,
-    taxAmount: document.tax_cost_currency,
-    terms: document.payment_terms,
-    totalPrice: document.retail_total_price_currency,
     shippingPrice: document.shipping_cost_currency,
     shipTo: {
       organizationName: document.ship_to.organization_name,
@@ -208,9 +204,27 @@ export const configureDocuments = (documents, requestIdentifier) => {
       organizationName: document.ship_from.organization_name,
       text: document.ship_from.text,
     },
+    subtotalPrice: document.retail_subtotal_price_currency,
+    taxAmount: document.tax_cost_currency,
+    terms: document.payment_terms,
+    totalPrice: document.retail_total_price_currency,
+  }
+}
+
+export const configureSOWs = (sows, requestIdentifier) => {
+  return sows?.map((sow) => ({
+    ...configureDocument(sow, requestIdentifier),
   }))
 }
 
+export const configurePO = (po, requestIdentifier) => ({
+  ...configureDocument(po, requestIdentifier),
+  turnaroundTime: po.turn_around_time.human,
+  poNumber: po.po_number,
+  relatedSOWIdentifier: po.proposal_ref?.identifier,
+  adPO: po.scientist_identifier,
+})
+
 const configureLineItems = (lineItems) => (lineItems.map(lineItem => ({
   id: lineItem.id,
   quantity: lineItem.quantity,

utils/api/requests.js

@@ -1,11 +1,12 @@
 import useSWR from 'swr'
 import {
   configureFiles,
-  configureDocuments,
   configureDynamicFormSchema,
   configureDynamicFormUiSchema,
   configureMessages,
+  configurePO,
   configureRequests,
+  configureSOWs,
 } from './configurations'
 import { fetcher, posting, updating } from './base'
 
@@ -45,7 +46,7 @@ export const useAllSOWs = (id, requestIdentifier, accessToken) => {
   const { data, error } = useSWR(id ? [`/quote_groups/${id}/proposals.json`, accessToken] : null)
   let allSOWs
   if (data) {
-    allSOWs = configureDocuments(data, requestIdentifier)
+    allSOWs = configureSOWs(data, requestIdentifier)
   }
 
   return {
@@ -55,6 +56,33 @@ export const useAllSOWs = (id, requestIdentifier, accessToken) => {
   }
 }
 
+// The name of this function is getAllPOs vs. useAllPOs. Since it is not a hook, it should not start with "use".
+export const getAllPOs = async (quotedWareId, uuid, requestIdentifier, accessToken) => {
+  try {
+    // TODO(summer-cook): eventually we can use the useSWRList hook here instead of mapping & calling the fetcher.
+    // This hook is actively being contributed to the swr repo, but the semantics of the work are still being debated.
+    // See https://github.com/vercel/swr/discussions/1988 for the RFC and https://github.com/vercel/swr/pull/2047 for the PR.
+    const data = await fetcher(`quote_groups/${uuid}/quoted_wares/${quotedWareId}/purchase_orders.json`, accessToken)
+    const configuredPOs = data.map(async (po) => {
+      const purchaseOrder = await fetcher(`quote_groups/${uuid}/quoted_wares/${quotedWareId}/purchase_orders/${po.id}.json`, accessToken)
+      return configurePO(purchaseOrder, requestIdentifier)
+    })
+    const allPOs = await Promise.all(configuredPOs).then(res => res)
+
+    return {
+      allPOs,
+      isLoadingPOs: !allPOs,
+      isPOError: false,
+    }
+  } catch (error) {
+    return {
+      allPOs: [],
+      isLoadingPOs: false,
+      isPOError: error,
+    }
+  }
+}
+
 export const useMessages = (requestUuid, accessToken) => {
   const { data, error, mutate } = useSWR(requestUuid ? [`/quote_groups/${requestUuid}/messages.json`, accessToken] : null)
   let messages