[BE/#94] PATCH /posts/{id} API 구현 #96

namewhat99 · 2023-11-21T06:17:33Z

이슈

koomin1227

전반 적으로 깔끔 하게 작성하신 것 같아 굿입니다.

koomin1227 · 2023-11-21T13:00:48Z

BE/src/post/post.service.ts

+
+    const isChangingImages = 'images' in updatePostDto; // images 가 존재여부 확인
+
+    try {


세세한 경우를 잘 나누어서 구현하신 것이 좋네요

koomin1227 · 2023-11-21T13:05:14Z

BE/src/post/post.service.ts

+    try {
+      await this.postRepository.update({ id: postId }, updatePostDto);
+      return true;
+    } catch {
+      return false;
+    }


여기에 try catch 문을 사용하셔서 결국에는 에러가 나도 위로 전파되지 않고 항상 true 또는 false를 반환하게 되는데updatePostById에 try catch 쓰는것이 의미가 없지 않나 하는 생각이 듭니다..

[BE] Feat : 게시물 수정 API 구현

301cfe7

namewhat99 added review code-review BE BE 관련 post-manage 게시글 관리 labels Nov 21, 2023

namewhat99 requested a review from koomin1227 November 21, 2023 06:17

namewhat99 self-assigned this Nov 21, 2023

[BE] Fix : 이미지 patch 에 대한 추가 처리 구현

62b6393

koomin1227 approved these changes Nov 21, 2023

View reviewed changes

namewhat99 and others added 2 commits November 21, 2023 22:40

[BE] Fix : postController 수정

e082784

Merge branch 'BE' into BE-patchPost-#94

eac58ee

koomin1227 merged commit 50d983c into BE Nov 21, 2023
1 check failed

koomin1227 deleted the BE-patchPost-#94 branch November 21, 2023 14:23


		const isChangingImages = 'images' in updatePostDto; // images 가 존재여부 확인

		try {