linting

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

@@ -22,7 +22,7 @@ import {
   statusBarBg,
   useMessages,
   useFiles,
-  useAllPOs,
+  getAllPOs,
   useAllSOWs,
   useOneRequest,
 } from '../../utils'
@@ -45,7 +45,7 @@ const Request = () => {
   const [isPOLoading, setIsPOLoading] = useState(false)
   useEffect(() => {
     const getPOsAsync = async () => {
-      const response = await useAllPOs(request?.quotedWareID, uuid, request?.identifier, session?.accessToken)
+      const response = await getAllPOs(request?.quotedWareID, uuid, request?.identifier, session?.accessToken)
       if (response.allPOs) {
         setAllPOs(response.allPOs)
       } else if (response.isPOError) {
@@ -55,7 +55,7 @@ const Request = () => {
       }
     }
     getPOsAsync()
-  }, [allPOs, isPOError]);
+  }, [allPOs, isPOError, request?.quotedWareID, uuid, request?.identifier, session?.accessToken])
 
   console.log(allPOs)
   const isLoading = isLoadingRequest || isLoadingSOWs || isLoadingFiles || isLoadingMessages || isPOLoading

utils/api/requests.js

@@ -55,12 +55,16 @@ export const useAllSOWs = (id, requestIdentifier, accessToken) => {
   }
 }
 
-// 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.
-export const useAllPOs = async (quotedWareId, uuid, requestIdentifier, accessToken) => {
+// The name of this function is getAllPOs vs. useAllPOs.
+// Since it is async, it is technically not a custom hook and according to linter should not start with use.
+export const getAllPOs = async (quotedWareId, uuid, requestIdentifier, accessToken) => {
   let allPOs
   let enhancedPOArray = []
   let arrayOfPOIds = []
-  try {
+  try { d
+    // 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 allPOData = await fetcher(`quote_groups/${uuid}/quoted_wares/${quotedWareId}/purchase_orders.json`, accessToken)
     allPOData && allPOData.map((po) => {arrayOfPOIds.push(po.id)})
     arrayOfPOIds && await Promise.all(arrayOfPOIds.map(async (poId) => {