Merge pull request #282 from TripInfoWeb/dev_diary

Refactor: 일기 수정 및 삭제 API를 Path Parameter 방식으로 변경
TripInfoWeb · Sep 10, 2024 · 550d618 · 550d618
2 parents 6c0db6c + 6de0f24
commit 550d618
Show file tree

Hide file tree

Showing 4 changed files with 62 additions and 57 deletions.
diff --git a/src/app/api/diary/[id]/route.ts b/src/app/api/diary/[id]/route.ts
@@ -0,0 +1,58 @@
+import { UpdateDiaryRequestDto } from "@/types/DiaryDto";
+import { revalidateTag } from "next/cache";
+import { NextRequest } from "next/server";
+
+/**
+ * 일기 수정
+ */
+export async function PUT(
+  request: NextRequest,
+  { params }: { params: { id: string } },
+) {
+  const cookie = request.cookies.get("access_token");
+  const body: UpdateDiaryRequestDto = await request.json();
+
+  const response = await fetch(
+    `${process.env.BACKEND_URL}/api/diary/${params.id}`,
+    {
+      method: "PUT",
+      headers: {
+        "Content-Type": "application/json",
+        Cookie: `${cookie?.name}=${cookie?.value}`,
+      },
+      body: JSON.stringify(body),
+      cache: "no-store",
+    },
+  );
+
+  if (!response.ok) {
+    throw new Error(response.statusText);
+  }
+
+  revalidateTag("getDiaryList");
+  revalidateTag(`getDiary/${params.id}`);
+  return response;
+}
+
+/**
+ * 일기 삭제
+ */
+export async function DELETE(
+  request: NextRequest,
+  { params }: { params: { id: string } },
+) {
+  const cookie = request.cookies.get("access_token");
+  const response = await fetch(
+    `${process.env.BACKEND_URL}/api/diary/${params.id}`,
+    {
+      method: "DELETE",
+      headers: {
+        Cookie: `${cookie?.name}=${cookie?.value}`,
+      },
+      cache: "no-store",
+    },
+  );
+
+  revalidateTag("getDiaryList");
+  return response;
+}
diff --git a/src/app/api/diary/route.ts b/src/app/api/diary/route.ts
@@ -1,4 +1,4 @@
-import { CreateDiaryRequestDto, UpdateDiaryRequestDto } from "@/types/DiaryDto";
+import { CreateDiaryRequestDto } from "@/types/DiaryDto";
 import { revalidateTag } from "next/cache";
 import { NextRequest } from "next/server";
 
@@ -20,55 +20,3 @@ export async function POST(request: NextRequest) {
   revalidateTag("getDiaryList");
   return response;
 }
-
-/**
- * 일기 수정
- */
-export async function PUT(request: NextRequest) {
-  const cookie = request.cookies.get("access_token");
-  const diaryId = request.nextUrl.searchParams.get("diaryId");
-  const body: UpdateDiaryRequestDto = await request.json();
-
-  const response = await fetch(
-    `${process.env.BACKEND_URL}/api/diary?diaryId=${diaryId}`,
-    {
-      method: "PUT",
-      headers: {
-        "Content-Type": "application/json",
-        Cookie: `${cookie?.name}=${cookie?.value}`,
-      },
-      body: JSON.stringify(body),
-      cache: "no-store",
-    },
-  );
-
-  if (!response.ok) {
-    throw new Error(response.statusText);
-  }
-
-  revalidateTag("getDiaryList");
-  revalidateTag(`getDiary/${diaryId}`);
-  return response;
-}
-
-/**
- * 일기 삭제
- */
-export async function DELETE(request: NextRequest) {
-  const cookie = request.cookies.get("access_token");
-  const diaryId = request.nextUrl.searchParams.get("diaryId");
-
-  const response = await fetch(
-    `${process.env.BACKEND_URL}/api/diary?diaryId=${diaryId}`,
-    {
-      method: "DELETE",
-      headers: {
-        Cookie: `${cookie?.name}=${cookie?.value}`,
-      },
-      cache: "no-store",
-    },
-  );
-
-  revalidateTag("getDiaryList");
-  return response;
-}
diff --git a/src/containers/diary/detail/DiaryDeleteModalContainer.tsx b/src/containers/diary/detail/DiaryDeleteModalContainer.tsx
@@ -16,7 +16,7 @@ const DiaryDeleteModalContainer = ({ diaryId, closeModal }: Props) => {
   const onDeleteClick = async () => {
     setLoading(true);
 
-    const response = await fetch(`/api/diary?diaryId=${diaryId}`, {
+    const response = await fetch(`/api/diary/${diaryId}`, {
       method: "DELETE",
       cache: "no-store",
     });

diff --git a/src/containers/diary/edit/DiaryEditorContainer.tsx b/src/containers/diary/edit/DiaryEditorContainer.tsx
@@ -110,7 +110,7 @@ const DiaryEditorContainer = ({ diaryData }: Props) => {
     setLoading(true);
 
     const response = await fetch(
-      `/api/diary?diaryId=${diaryData.diaryContentResponse.diaryId}`,
+      `/api/diary/${diaryData.diaryContentResponse.diaryId}`,
       {
         method: "PUT",
         headers: {
@@ -127,8 +127,7 @@ const DiaryEditorContainer = ({ diaryData }: Props) => {
       throw new Error(response.statusText);
     }
 
-    const diaryId = await response.text();
-    router.push(`/diary/${diaryId}`);
+    router.push(`/diary/${diaryData.diaryContentResponse.diaryId}`);
     router.refresh();
   };