more correct comment explanation

alishaevn · alishaevn · commit 953ffe9acf52 · 2023-03-07T17:19:27.000-06:00
diff --git a/utils/api/requests.js b/utils/api/requests.js
@@ -56,9 +56,7 @@ export const useAllSOWs = (id, 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) => {
+// The name of this function is getAllPOs vs. useAllPOs. Since it is not a hook, it should not start with "use".
   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.

Original file line number	Diff line number	Diff line change
`@@ -56,9 +56,7 @@ export const useAllSOWs = (id, requestIdentifier, accessToken) => {`
`56`	`56`	`}`
`57`	`57`	`}`
`58`	`58`
`59`		`-// The name of this function is getAllPOs vs. useAllPOs.`
`60`		`-// Since it is async, it is technically not a custom hook and according to linter should not start with use.`
`61`		`-export const getAllPOs = async (quotedWareId, uuid, requestIdentifier, accessToken) => {`
	`59`	`+// The name of this function is getAllPOs vs. useAllPOs. Since it is not a hook, it should not start with "use".`
`62`	`60`	`try {`
`63`	`61`	`// TODO(summer-cook): eventually we can use the useSWRList hook here instead of mapping & calling the fetcher.`
`64`	`62`	`// This hook is actively being contributed to the swr repo, but the semantics of the work are still being debated.`