프로덕션 배포를 위해 머지합니다. (#159)

* fix: 로그인 안내문구 수정 * chore: 서명 미업로드 설명 추가 * fix: 로그인 안내문구 수정 (#146) * fix: department null인 경우 예외처리 (#148) * [FIX] 심사 내용 없을때 등록 안되도록 수정 (#158) * fix: 심사시 심사의견파일 혹은 심사의견 필수 수정 * fix: 일괄등록 안내문구 추가 * fix: 최종심사 모달창 문구 수정 * fix: 관리자 심사 수정시 심사 내용 필수로 수정 * fix: 학생 심사 결과 페이지 심사의견 볼수있도록 수정 (#156) * feat: 학생 개별삭제, 교수 개별삭제 기능 추가 (#150) * fix: department null인 경우 예외처리 (#151) * fix: 교수/학생 로그인하기 router.refresh 추가 (#152) * fix: PhaseReady 시스템 단계 기간 주석처리 (#153) * fix: PhaseEditFormatRow Date 오류수정 (#154) * fix: 수정지시사항 단계 제외 오류 해결 및 학과관리 페이지로 이동 (#155)
SystemConsultantGroup · May 3, 2024 · 484a2a0 · 484a2a0
1 parent 8cb6142
commit 484a2a0
Show file tree

Hide file tree

Showing 26 changed files with 326 additions and 170 deletions.
diff --git a/src/api/apiRoute.ts b/src/api/apiRoute.ts
@@ -56,7 +56,7 @@ export const API_ROUTES = {
       `/students/${studentId}/headReviewer/${reviewerId}`, // PUT: 학생 심사위원장 배정
     deleteReviewer: (studentId: ApiId, reviewerId: ApiId) =>
       `/students/${studentId}/reviewers/${reviewerId}`, // PUT: 학생 심사위원 배정 취소
-    delete: () => "/students", // DELETE: 학생 삭제
+    delete: (studentId?: ApiId) => `/students/${studentId ?? ""}`, // DELETE: 학생 삭제
   },
   review: {
     // 학생 본인의 논문 조회

diff --git a/src/app/admin/results/[thesisId]/AdminReviewContent.tsx b/src/app/admin/results/[thesisId]/AdminReviewContent.tsx
@@ -8,7 +8,7 @@ import { AdminReviewResponse, ThesisReview, UpdateReviewRequestBody } from "@/ap
 import { transactionTask } from "@/api/_utils/task";
 import { API_ROUTES } from "@/api/apiRoute";
 import { ApiDownloadButton } from "@/components/common/Buttons";
-import { showNotificationSuccess } from "@/components/common/Notifications";
+import { showNotificationError, showNotificationSuccess } from "@/components/common/Notifications";
 import SectionTitle from "@/components/common/SectionTitle";
 import {
   BasicRow,
@@ -159,12 +159,26 @@ function ModalContent({ open, setOpen, data, current }: ModalProps) {
           task.onComplete(() => setLoading(false));
 
           let fileUUID;
-          if (reviewFile) {
+          if (reviewFile && commentType === "심사 의견 파일") {
             fileUUID = (await uploadFile(reviewFile)).uuid;
           } else if (current.file) {
             fileUUID = current.file.uuid ?? undefined;
           }
 
+          if (thesis === "UNEXAMINED" || presentation === "UNEXAMINED") {
+            showNotificationError({ message: "합격 여부를 선택해주세요." });
+            return;
+          }
+
+          if (
+            commentType === undefined ||
+            (commentType === "심사 의견" && (comment === undefined || !comment)) ||
+            (commentType === "심사 의견 파일" && fileUUID === undefined)
+          ) {
+            showNotificationError({ message: "심사 의견이나 심사 의견 파일을 첨부해주세요." });
+            return;
+          }
+
           await ClientAxios.put(
             current.isFinal
               ? API_ROUTES.review.final.put(current.id)

diff --git a/src/app/admin/reviews/[thesisId]/AdminReviewListContent.tsx b/src/app/admin/reviews/[thesisId]/AdminReviewListContent.tsx
@@ -6,7 +6,7 @@ import { ClientAxios } from "@/api/ClientAxios";
 import { AdminReviewResponse, ThesisReview, UpdateReviewRequestBody } from "@/api/_types/reviews";
 import { transactionTask } from "@/api/_utils/task";
 import { API_ROUTES } from "@/api/apiRoute";
-import { showNotificationSuccess } from "@/components/common/Notifications";
+import { showNotificationError, showNotificationSuccess } from "@/components/common/Notifications";
 import SectionTitle from "@/components/common/SectionTitle";
 import {
   BasicRow,
@@ -91,12 +91,26 @@ function ModalContent({ open, setOpen, data, current }: ModalProps) {
           task.onComplete(() => setLoading(false));
 
           let fileUUID;
-          if (reviewFile) {
+          if (reviewFile && commentType === "심사 의견 파일") {
             fileUUID = (await uploadFile(reviewFile)).uuid;
           } else if (current.file) {
             fileUUID = current.file.uuid ?? undefined;
           }
 
+          if (thesis === "UNEXAMINED" || presentation === "UNEXAMINED") {
+            showNotificationError({ message: "합격 여부를 선택해주세요." });
+            return;
+          }
+
+          if (
+            commentType === undefined ||
+            (commentType === "심사 의견" && (comment === undefined || !comment)) ||
+            (commentType === "심사 의견 파일" && fileUUID === undefined)
+          ) {
+            showNotificationError({ message: "심사 의견이나 심사 의견 파일을 첨부해주세요." });
+            return;
+          }
+
           await ClientAxios.put(
             data.stage === "REVISION"
               ? API_ROUTES.review.revision.put(current.id)

diff --git a/src/app/prof/final/[id]/ProfessorFinalForm.tsx b/src/app/prof/final/[id]/ProfessorFinalForm.tsx
@@ -87,8 +87,8 @@ export function ProfessorFinalForm({
       }했습니다.`,
     });
 
+    router.push("/prof/final");
     router.refresh();
-    router.push("../final");
   });
 
   return (
@@ -100,10 +100,10 @@ export function ProfessorFinalForm({
           handleSubmit(input);
         } else {
           const hasCommentFile = previous.reviewFile
-            ? values.commentFile === null
+            ? values.commentFile !== null
             : !!values.commentFile;
-          if (values.comment === "" && hasCommentFile) {
-            showNotificationError({ message: <>심사 의견이나 심사 의견 파일을 첨부해주세요.</> });
+          if (!values.comment && !hasCommentFile) {
+            showNotificationError({ message: "심사 의견이나 심사 의견 파일을 첨부해주세요." });
             return;
           }
           setShowConfirmDialog(true);
@@ -130,6 +130,7 @@ export function ProfessorFinalForm({
         onClose={() => {
           setShowConfirmDialog(false);
         }}
+        isFinal
       />
     </form>
   );

diff --git a/src/app/prof/review/[id]/ProfessorReviewForm.tsx b/src/app/prof/review/[id]/ProfessorReviewForm.tsx
@@ -69,12 +69,20 @@ export function ProfessorReviewForm({
 
     const isPending = input.thesis === "PENDING" || input.presentation === "PENDING";
     let fileUUID;
-    if (input.commentFile) {
+    if (input.commentFile && commentType === "심사 의견 파일") {
       fileUUID = (await uploadFile(input.commentFile)).uuid;
     } else if (input.commentFile !== null) {
       fileUUID = previous.reviewFile?.uuid ?? undefined;
     }
 
+    if (
+      (commentType === "심사 의견" && !input.comment) ||
+      (commentType === "심사 의견 파일" && fileUUID === undefined)
+    ) {
+      showNotificationError({ message: "심사 의견이나 심사 의견 파일을 첨부해주세요." });
+      return;
+    }
+
     await ClientAxios.put(
       API_ROUTES.review.put(reviewId),
       {
@@ -98,8 +106,8 @@ export function ProfessorReviewForm({
 
     // TODO: 불필요한 fetch를 추가하긴 하지만, 이것 말고 적당한 방법이 있는지 모르겠음...
     // https://github.com/vercel/next.js/discussions/54075 참고: 현재는 클라이언트측 Router Cache를 완전히 비활성화할 방법이 없음
+    router.push("/prof/review");
     router.refresh();
-    router.push("../review");
   });
 
   return (
@@ -111,10 +119,10 @@ export function ProfessorReviewForm({
           handleSubmit(input);
         } else {
           const hasCommentFile = previous.reviewFile
-            ? values.commentFile === null
+            ? values.commentFile !== null
             : !!values.commentFile;
-          if (values.comment === "" && hasCommentFile) {
-            showNotificationError({ message: <>심사 의견이나 심사 의견 파일을 첨부해주세요.</> });
+          if (!values.comment && !hasCommentFile) {
+            showNotificationError({ message: "심사 의견이나 심사 의견 파일을 첨부해주세요." });
             return;
           }
           setShowConfirmDialog(true);
@@ -134,8 +142,8 @@ export function ProfessorReviewForm({
         review={{
           thesis: values.thesis,
           presentation: values.presentation,
-          comment: values.comment,
-          commentFile: values.commentFile?.name ?? null,
+          comment: commentType === "심사 의견" ? values.comment : "",
+          commentFile: commentType === "심사 의견 파일" ? values.commentFile?.name ?? null : null,
         }}
         opened={showConfirmDialog}
         onConfirm={() => handleSubmit(values)}

diff --git a/src/app/student/result/page.tsx b/src/app/student/result/page.tsx
@@ -12,12 +12,23 @@ import { UserResponse } from "@/api/_types/user";
 export default async function StudentResultPage() {
   const { token } = await AuthSSR({ userType: "STUDENT" });
   const user = (await fetcher({ url: API_ROUTES.user.get(), token })) as UserResponse;
-  const { "0": pre, "1": main } = (await fetcher({ url: API_ROUTES.review.getMe(), token })) as {
-    "0": MyReviewResponse;
-    "1": MyReviewResponse;
+  const response = (await fetcher({ url: API_ROUTES.review.getMe(), token })) as {
+    [key: string]: MyReviewResponse;
   };
 
-  const thesisRes: MyReviewResponse = user.currentPhase === "PRELIMINARY" ? pre : main;
+  let main: MyReviewResponse;
+  let pre: MyReviewResponse;
+
+  for (let i = 0; i < 3; i += 1) {
+    if (response[i.toString()]?.stage === "MAIN") {
+      main = response[i.toString()];
+    } else if (response[i.toString()]?.stage === "PRELIMINARY") {
+      pre = response[i.toString()];
+    }
+  }
+
+  const thesisRes: MyReviewResponse = user.currentPhase === "PRELIMINARY" ? pre! : main!;
+
   const thesisInfo: ThesisInfoData = {
     title: thesisRes.title,
     stage: thesisRes.stage,

diff --git a/src/components/common/AppShell/AppShell.tsx b/src/components/common/AppShell/AppShell.tsx
@@ -14,18 +14,18 @@ function AppShell({ children }: Props) {
   const { user } = useAuth();
   const pathname = usePathname();
   const disabledAppShell = user?.type === null || pathname === "/login";
-
+  const modificationFlag =
+    user?.type === "STUDENT"
+      ? user?.department.modificationFlag && user.currentPhase !== "PRELIMINARY"
+      : false;
   return (
     <MantineAppShell
       padding={40}
       navbar={{ width: 300, breakpoint: "sm" }}
       styles={{ main: { background: "var(--mantine-color-main-background)" } }}
       disabled={disabledAppShell}
     >
-      <Navbar
-        userType={user?.type}
-        modificationFlag={user?.department.modificationFlag && user.currentPhase !== "PRELIMINARY"}
-      />
+      <Navbar userType={user?.type} modificationFlag={modificationFlag} />
       <MantineAppShell.Main>{children}</MantineAppShell.Main>
     </MantineAppShell>
   );

diff --git a/src/components/common/AppShell/_elements/Navbar.tsx b/src/components/common/AppShell/_elements/Navbar.tsx
@@ -30,9 +30,7 @@ function Navbar({ userType, modificationFlag }: Props) {
       <AppShell.Section>
         <Profile />
       </AppShell.Section>
-      {userType && modificationFlag !== undefined && (
-        <NavbarList userType={userType} modificationFlag={modificationFlag} />
-      )}
+      {userType && <NavbarList userType={userType} modificationFlag={modificationFlag} />}
     </AppShell.Navbar>
   );
 }

diff --git a/src/components/common/AppShell/_elements/NavbarList.tsx b/src/components/common/AppShell/_elements/NavbarList.tsx
@@ -11,13 +11,13 @@ import {
 
 interface Props {
   userType: Role;
-  modificationFlag: boolean;
+  modificationFlag?: boolean;
 }
 
 const USER_TYPE_NAVBAR_LIST = {
   ADMIN: ADMIN_NAVBAR_LIST,
   PROFESSOR: PROF_NAVBAR_LIST,
-  STUDENT: (modificationFlag: boolean) => STUDENT_NAVBAR_LIST(modificationFlag),
+  STUDENT: (modificationFlag?: boolean) => STUDENT_NAVBAR_LIST(modificationFlag),
 };
 
 function NavbarList({ userType, modificationFlag }: Props) {

diff --git a/src/components/common/AppShell/_elements/constant/navbarList.tsx b/src/components/common/AppShell/_elements/constant/navbarList.tsx
@@ -78,7 +78,7 @@ export const ADMIN_NAVBAR_LIST: Props[] = [
     icon: <IconSettings size="24" stroke={1} />,
   },
 ];
-export const STUDENT_NAVBAR_LIST = (modificationFlag: boolean) => {
+export const STUDENT_NAVBAR_LIST = (modificationFlag?: boolean) => {
   const navbarList: Props[] = [
     { label: "메인", href: "/", icon: <IconHome size="24" stroke={1} /> },
     { label: "논문 투고", href: "/student/write", icon: <IconFileText size="24" stroke={1} /> },

diff --git a/src/components/pages/AdminExcelRegister/AdminExcelRegister.tsx b/src/components/pages/AdminExcelRegister/AdminExcelRegister.tsx
@@ -75,7 +75,13 @@ function AdminExcelRegister({ isProf = false }: Props) {
     <Stack gap={0}>
       <TitleRow
         title={isProf ? "교수 기본 정보" : "학생 기본 정보"}
-        subString="* 이미 등록된 중복 아이디의 경우 데이터가 수정됩니다."
+        subString={
+          <>
+            <>* 이미 등록된 중복 아이디의 경우 데이터가 수정됩니다.</>
+            <br />
+            <>* 한 번에 많은 사용자를 업로드할 시 로딩 시간이 길어질 수 있습니다.</>
+          </>
+        }
       />
       <RowGroup>
         <NoticeRow text="첨부파일 우측 상단의 안내사항을 참고하시기 바랍니다." />