Fix/allow user edits for open review request (#1587)

* Chore: add separate check for disabled write actions

* fix: update site header behaviour

src/layouts/layouts/SiteEditLayout/SiteEditHeader.tsx

@@ -28,11 +28,10 @@ import { useDirtyFieldContext } from "contexts/DirtyFieldContext"
 import { useLoginContext } from "contexts/LoginContext"
 
 import { useStagingUrl } from "hooks/settingsHooks"
-import { useGetReviewRequests } from "hooks/siteDashboardHooks"
 
 import { ReviewRequestModal } from "layouts/ReviewRequest"
 
-import { isWriteActionsDisabled } from "utils/reviewRequests"
+import { doesOpenReviewRequestExist } from "utils/reviewRequests"
 
 import { NavImage } from "assets"
 
@@ -57,7 +56,7 @@ export const SiteEditHeader = (): JSX.Element => {
   const isGithubUser = !!userId
   const history = useHistory()
 
-  const isWriteDisabled = isWriteActionsDisabled(siteName)
+  const hasOpenReviewRequest = doesOpenReviewRequestExist(siteName)
 
   const onBackButtonClick = () => {
     const redirPath = isGithubUser ? "/sites" : `/sites/${siteName}/dashboard`
@@ -117,7 +116,7 @@ export const SiteEditHeader = (): JSX.Element => {
               id="isomer-workspace-feature-tour-step-1"
               leftIcon={<Icon as={BiCheckCircle} fontSize="1.25rem" />}
               onClick={onReviewRequestModalOpen}
-              isDisabled={isWriteDisabled}
+              isDisabled={hasOpenReviewRequest}
             >
               Request a Review
             </Button>

src/utils/reviewRequests.ts

@@ -1,6 +1,6 @@
 import { useGetReviewRequests } from "hooks/siteDashboardHooks"
 
-export const isWriteActionsDisabled = (siteName: string) => {
+export const doesOpenReviewRequestExist = (siteName: string) => {
   const { data: reviewRequests, isLoading, isError } = useGetReviewRequests(
     siteName
   )
@@ -13,3 +13,17 @@ export const isWriteActionsDisabled = (siteName: string) => {
 
   return isLoading || isError || isReviewRequestPending
 }
+
+export const isWriteActionsDisabled = (siteName: string) => {
+  const { data: reviewRequests, isLoading, isError } = useGetReviewRequests(
+    siteName
+  )
+
+  // Note: if PR is in APPROVED status, it will auto-redirect to dashboard as no edits should happen
+  // But have added here to be explicit of the status checks
+  const isReviewRequestPending = reviewRequests?.some(
+    (request) => request.status === "APPROVED"
+  )
+
+  return isLoading || isError || isReviewRequestPending
+}