Merge pull request #312 from Nexters/develop

[운영 배포] 임시 공연 입력 폼 비활성화 로직 해제 복구

Author: Puterism

apps/admin/src/pages/ShowInfoPage/index.tsx

@@ -9,7 +9,7 @@ import {
 } from '@boolti/api';
 import { Button, Drawer, ShowPreview, useConfirm, useToast } from '@boolti/ui';
 import { format } from 'date-fns';
-import { useCallback, useEffect, useRef, useState } from 'react';
+import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import { useForm } from 'react-hook-form';
 import { useParams } from 'react-router-dom';
 
@@ -57,18 +57,21 @@ const ShowInfoPage = () => {
   const editShowInfoMutation = useEditShowInfo();
   const uploadShowImageMutation = useUploadShowImage();
   const isNonTicketingShow = !showSalesInfo?.salesEndTime && !showSalesInfo?.salesStartTime;
-  // const isFormDisabled = !isNonTicketingShow && show?.isEnded;
-  const isFormDisabled = false
-
-  // const isSaveButtonDisabled = useMemo(
-  //   () =>
-  //     !showBasicInfoForm.formState.isValid ||
-  //     !showDetailInfoForm.formState.isValid ||
-  //     imageFiles.length === 0 ||
-  //     (isFormDisabled),
-  //   [imageFiles.length, isFormDisabled, showBasicInfoForm.formState.isValid, showDetailInfoForm.formState.isValid],
-  // );
-  const isSaveButtonDisabled = false
+
+  const isSaveButtonDisabled = useMemo(
+    () =>
+      !showBasicInfoForm.formState.isValid ||
+      !showDetailInfoForm.formState.isValid ||
+      imageFiles.length === 0 ||
+      (!isNonTicketingShow && show?.isEnded),
+    [
+      imageFiles.length,
+      isNonTicketingShow,
+      show?.isEnded,
+      showBasicInfoForm.formState.isValid,
+      showDetailInfoForm.formState.isValid,
+    ],
+  );
 
   const toast = useToast();
   const confirm = useConfirm();
@@ -89,12 +92,12 @@ const ShowInfoPage = () => {
       return await uploadShowImageMutation.mutateAsync(newImageFiles);
     })();
 
-    // const [isValidShowBasicInfoFormInputs, isValidShowDetailInfoFormInputs] = await Promise.all([
-    //   showBasicInfoForm.trigger(),
-    //   showDetailInfoForm.trigger(),
-    // ]);
+    const [isValidShowBasicInfoFormInputs, isValidShowDetailInfoFormInputs] = await Promise.all([
+      showBasicInfoForm.trigger(),
+      showDetailInfoForm.trigger(),
+    ]);
 
-    // if (!isValidShowBasicInfoFormInputs || !isValidShowDetailInfoFormInputs) return;
+    if (!isValidShowBasicInfoFormInputs || !isValidShowDetailInfoFormInputs) return;
 
     const showBasicInfoFormInputs = showBasicInfoForm.getValues();
     const showDetailInfoFormInputs = showDetailInfoForm.getValues();
@@ -258,7 +261,7 @@ const ShowInfoPage = () => {
             form={showBasicInfoForm}
             imageFiles={imageFiles}
             isNonTicketingShow={isNonTicketingShow}
-            disabled={isFormDisabled}
+            disabled={!isNonTicketingShow && show.isEnded}
             onDropImage={(acceptedFiles) => {
               setImageFiles((prevImageFiles) => [
                 ...prevImageFiles,
@@ -282,7 +285,7 @@ const ShowInfoPage = () => {
         <Styled.ShowInfoFormContent style={{ marginBottom: '48px' }}>
           <ShowDetailInfoFormContent
             form={showDetailInfoForm}
-            disabled={isFormDisabled}
+            disabled={!isNonTicketingShow && show.isEnded}
           />
         </Styled.ShowInfoFormContent>
         <Styled.ShowInfoFormContent>