Skip to content

Feature/1 comment api & Bug Fix#37

Merged
sungchaelee merged 22 commits into
IssueDiver:devfrom
sungchaelee:feature/1-comment-api
Aug 29, 2025
Merged

Feature/1 comment api & Bug Fix#37
sungchaelee merged 22 commits into
IssueDiver:devfrom
sungchaelee:feature/1-comment-api

Conversation

@sungchaelee
Copy link
Copy Markdown
Contributor

연관된 이슈

#1 , #34


작업 내용

댓글 CRUD 개발 & Unit Test 코드 작성

CommentResponse에서 트리 변환 중 부모 DTO의 children이 null인데 .add()를 호출해서 NPE가 났음.
-> CommentResponse가 항상 빈리스트를 보장하게 만듦.

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).
  • main branch가 아닌 dev branch에 PR 요청을 했습니다. (main branch에 바로 PR&merge하지 않기).

…eature/1-comment-api

# Conflicts:
#	.gitignore
…eature/1-comment-api

# Conflicts:
#	build.gradle
#	gradlew
#	src/main/java/com/issueDive/controller/CommentController.java
#	src/main/java/com/issueDive/dto/CommentResponse.java
#	src/main/java/com/issueDive/dto/CreateCommentRequest.java
#	src/main/java/com/issueDive/dto/UpdateCommentRequest.java
#	src/main/java/com/issueDive/entity/Comment.java
#	src/main/java/com/issueDive/exception/CommentNotFoundException.java
#	src/main/java/com/issueDive/repository/CommentRepository.java
#	src/main/java/com/issueDive/service/CommentService.java
@sungchaelee sungchaelee reopened this Aug 29, 2025
…eature/1-comment-api

# Conflicts:
#	src/main/java/com/issueDive/exception/GlobalExceptionHandler.java
#	src/main/java/com/issueDive/exception/UserNotFoundException.java
…eature/1-comment-api

# Conflicts:
#	src/main/java/com/issueDive/exception/GlobalExceptionHandler.java
@sungchaelee sungchaelee reopened this Aug 29, 2025
@sungchaelee sungchaelee merged commit a018ad8 into IssueDiver:dev Aug 29, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant