improvements: address feedback

backend/src/ee/services/access-approval-request/access-approval-request-service.ts

@@ -380,9 +380,12 @@ export const accessApprovalRequestServiceFactory = ({
       );
 
       // reset review progress
-      await accessApprovalRequestReviewerDAL.delete({
-        requestId
-      });
+      await accessApprovalRequestReviewerDAL.delete(
+        {
+          requestId
+        },
+        tx
+      );
 
       const requesterFullName = `${requestedByUser.firstName} ${requestedByUser.lastName}`;
       const editorFullName = `${editedByUser.firstName} ${editedByUser.lastName}`;

frontend/src/hooks/api/accessApproval/mutation.tsx

@@ -149,7 +149,7 @@ export const useUpdateAccessRequest = () => {
     },
     onSuccess: (_, { projectSlug }) => {
       queryClient.invalidateQueries({
-        queryKey: accessApprovalKeys.getAccessApprovalPolicies(projectSlug)
+        queryKey: accessApprovalKeys.getAccessApprovalRequests(projectSlug)
       });
     }
   });

frontend/src/hooks/api/accessApproval/queries.tsx

@@ -24,7 +24,7 @@ export const accessApprovalKeys = {
     envSlug?: string,
     requestedBy?: string,
     bypassReason?: string
-  ) => [{ projectSlug, envSlug, requestedBy, bypassReason }, "access-approvals-requests"] as const,
+  ) => ["access-approvals-requests", projectSlug, envSlug, requestedBy, bypassReason] as const,
   getAccessApprovalRequestCount: (projectSlug: string, policyId?: string) =>
     [{ projectSlug }, "access-approval-request-count", ...(policyId ? [policyId] : [])] as const
 };

frontend/src/hooks/api/accessApproval/types.ts

@@ -104,7 +104,7 @@ export type TAccessApprovalRequest = {
 
   note?: string;
   editNote?: string;
-  editedByUserId: string;
+  editedByUserId?: string;
 };
 
 export type TAccessApproval = {

frontend/src/pages/secret-manager/SecretApprovalsPage/components/AccessApprovalRequest/AccessApprovalRequest.tsx

@@ -597,7 +597,7 @@ export const AccessApprovalRequest = ({
               // we shouldn't be using state like this...
               handleSelectRequest({
                 ...selectedRequest,
-                isTemporary: true,
+                isTemporary: request.isTemporary,
                 temporaryRange: request.temporaryRange,
                 reviewers: []
               });