[Chore] 게시글 관련 API 고도화 #31

yongbin97 · 2024-09-26T12:05:52Z

💡 연관된 이슈

📝 작업 내용

게시물 목록 조회 - pagination, search 기능 추가
게시물 상세 조회 - 비로그인 유저 예외 처리
요약 요청 - keyword 추가 및 프롬프트 수정

💬 리뷰 요구 사항

Dto관련해서 리팩토링한 코드와 충돌이 많이 일어나서 수정하느라 업데이트 된 파일이 많습니다.

junseoplee

고생하셨습니다! 컨벤션 및 간단한 의견을 남겼습니다.
그리고 pr 제목은 이슈와 같게 Feature로 변경하거나 Refactor로 바꾸는 게 좋을 거 같습니다.
chore는 로직 변경 및 코드 변경 없이 간단한 설정을 바꾸거나 할 때 사용하는 느낌이 강하다 생각합니다.

커밋 뒤에 해당 이슈 번호 붙여주세요 이미 푸쉬 된 건 머지할 때 메시지에 적용 부탁드립니다

깃허브 액션이 실패했었는데 제가 다시 돌리니 정상 적용됐네요! 가끔 인터넷 끊기거나 하면 이러더라구요

junseoplee · 2024-09-26T15:45:23Z

src/main/java/project/backend/business/post/PostService.java

 import project.backend.common.error.CustomException;
 import project.backend.common.error.ErrorCode;
 import project.backend.entity.post.Post;
 import project.backend.entity.user.User;
+import project.backend.repository.post.PostSpecification;
+


개행 1라인만 하도록 해주세요

수정했습니다.

junseoplee · 2024-09-26T15:49:15Z

src/main/java/project/backend/business/post/PostService.java

+                     .and(PostSpecification.getSearch(postListServiceRequest.getSearch()))
+                     .and(PostSpecification.getActivated());
+
+    PageRequest pageRequest = PageRequest.of(postListServiceRequest.getPage(), 10,


매직 넘버 상수화 부탁드립니다

수정했습니다

junseoplee · 2024-09-26T15:52:23Z

src/main/java/project/backend/business/post/PostService.java

+  public CreateUpdatePostResponse updatePostDetail(Long userId, Long postId,
+      PostDetailDto postDetailDto) {
+    Post post = postReader.read(userId, postId);
+    postManager.updatePost(post, postDetailDto);
+    Long id = post.getId();
+
+    return new CreateUpdatePostResponse(id);
+  }
+
+  public void deletePostDetail(Long userId, Long postId) {


updatePost가 아닌 PostDetail인 이유가 있을까요? Post와 어떤 게 다를까요?

게시물 상세 페이지에서 사용되는 컬럼 값들을 모아놓은 DTO입니다.
Update 뿐만 아니라 조회 시에도 사용되는 Dto입니다.
불필요한 컬럼들은 제외한 값입니다.

junseoplee · 2024-09-26T15:57:02Z

src/main/java/project/backend/business/post/implement/PostManager.java

+    post.setActivated(Boolean.FALSE);
+    postRepository.save(post);
+  }
+


불필요한 개행이네요

수정했습니다.

junseoplee · 2024-09-26T15:57:44Z

src/main/java/project/backend/business/post/implement/PostReader.java

 import project.backend.repository.post.PostRepository;
-import project.backend.entity.user.User;
+


불필요한 개행이네요

수정했습니다.

junseoplee · 2024-09-26T16:40:53Z

src/main/java/project/backend/repository/post/PostRepository.java

-  List<Post> findAllByUserIdAndStatus(Long userId, PostStatus status);
-
-  Post findPostByIdAndUserAndStatus(Long postId, User user, PostStatus status);
+  Post findByIdAndUserIdAndActivatedTrue(Long postId, Long userId);



불필요한 개행이 있어요

수정했습니다

junseoplee · 2024-09-26T16:42:14Z

src/main/java/project/backend/repository/post/PostSpecification.java

junseoplee · 2024-09-26T16:42:27Z

src/main/java/project/backend/repository/post/PostSpecification.java

+    return (root, query, criteriaBuilder) -> criteriaBuilder.equal(root.get("activated"),
+        Boolean.TRUE);
+  }
+


불필요한 개행이 있어요

수정했습니다.

junseoplee · 2024-09-26T16:45:01Z

src/main/java/project/backend/repository/tag/TagRepository.java

@@ -11,17 +11,17 @@
 public interface TagRepository extends JpaRepository<Tag, Long> {


시작은 개행으로 부탁드립니다

그리고 IDE에서 자동으로 관련있는 라이브러리끼리 모아두는데
이렇게 따로 직접 import를 구분하여 두시는 이유가 있을까요?

따로 없습니다. 수정했습니다.

junseoplee · 2024-09-26T16:47:34Z

src/main/java/project/backend/repository/tag/TagRepository.java

    List<String> findTagNamesByPostId(Long postId);

    @Query("SELECT t " +
            "FROM Tag AS t JOIN PostTag AS pt ON t = pt.tag " +
-            "WHERE pt.post.id = :postId ")
+            "WHERE pt.post.id = :postId AND t.activated = true")
    List<Tag> findAllByPostId(Long postId);

    List<Tag> findAllByNameIn(List<String> names);


아래 불필요한 개행도 삭제 부탁드립니다

수정했습니다

junseoplee · 2024-09-27T08:20:33Z

src/main/java/project/backend/business/post/implement/PostManager.java

@@ -31,21 +31,28 @@ public Long createTempPost(User user, String url, String summary) {
  }

  @Transactional


@transactional 은 상위 PostService에서 사용해주세요

수정했습니다~

yongbin97 added 6 commits September 23, 2024 23:00

chore: 게시물 목록 조회 - 쿼리 동적할당 (#26)

94e2eb4

chore: merge develop into branch

50b4aa5

feat: post list api pagination

a5c3158

chore: post list api search 기능 추가

d733c3a

chore: post detail api 비로그인 유저 예외처리

f592a04

chore: 요약 요청 API 고도화

8f3f509

yongbin97 added the ✨ Feature 기능 개발 label Sep 26, 2024

yongbin97 requested a review from junseoplee September 26, 2024 12:05

yongbin97 self-assigned this Sep 26, 2024

yongbin97 added 2 commits September 26, 2024 21:25

fix: 충돌 해결

a8bbc14

fix: security 설정 값 수정

7cac4f8

yongbin97 closed this Sep 26, 2024

yongbin97 added 2 commits September 26, 2024 23:03

feat: 게시물 삭제 (#26)

593c0b6

chore: activated true만 조회하기

8289506

yongbin97 reopened this Sep 26, 2024

junseoplee reviewed Sep 26, 2024

View reviewed changes

junseoplee reviewed Sep 27, 2024

View reviewed changes

yongbin97 added 4 commits September 27, 2024 22:08

refactor: 불필요한 코드 삭제

1cc0cf8

refactor: transactional 위치 수정

0226e2b

refactor: 불필요한 개행 삭제 및 컨벤션 수정

8541a07

refactor: null 처리 수정

149be57

yongbin97 merged commit d30560f into develop Sep 27, 2024
1 check passed

junseoplee deleted the feature/26-post-api branch September 27, 2024 15:06

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Chore] 게시글 관련 API 고도화 #31

[Chore] 게시글 관련 API 고도화 #31

yongbin97 commented Sep 26, 2024 •

edited

Loading

junseoplee left a comment •

edited

Loading

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 26, 2024

yongbin97 Sep 27, 2024

junseoplee Sep 27, 2024

yongbin97 Sep 27, 2024

		import project.backend.repository.post.PostRepository;
		import project.backend.entity.user.User;

		@@ -11,17 +11,17 @@
		public interface TagRepository extends JpaRepository<Tag, Long> {

		@@ -31,21 +31,28 @@ public Long createTempPost(User user, String url, String summary) {
		}

		@Transactional

[Chore] 게시글 관련 API 고도화 #31

[Chore] 게시글 관련 API 고도화 #31

Conversation

yongbin97 commented Sep 26, 2024 • edited Loading

💡 연관된 이슈

📝 작업 내용

💬 리뷰 요구 사항

junseoplee left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yongbin97 commented Sep 26, 2024 •

edited

Loading

junseoplee left a comment •

edited

Loading