Skip to content

20260223 #26 폴더 crud 및 폴더 장소 관리 api 구현 필요#33

Merged
Cassiiopeia merged 3 commits intomainfrom
20260223_#26_폴더_CRUD_및_폴더_장소_관리_API_구현_필요
Feb 23, 2026

Hidden character warning

The head ref may contain hidden characters: "20260223_#26_\ud3f4\ub354_CRUD_\ubc0f_\ud3f4\ub354_\uc7a5\uc18c_\uad00\ub9ac_API_\uad6c\ud604_\ud544\uc694"
Merged

20260223 #26 폴더 crud 및 폴더 장소 관리 api 구현 필요#33
Cassiiopeia merged 3 commits intomainfrom
20260223_#26_폴더_CRUD_및_폴더_장소_관리_API_구현_필요

Conversation

@Cassiiopeia
Copy link
Contributor

@Cassiiopeia Cassiiopeia commented Feb 23, 2026

✨ 변경 사항


✅ 테스트


  • 수동 테스트 완료
  • 테스트 코드 완료

Summary by CodeRabbit

릴리스 노트

  • 버그 수정

    • 폴더 및 기본 폴더에 장소 추가할 때 소프트삭제된 항목 복구 처리 개선
  • 기능 변경

    • 북마크 수정 시 폴더 변경 기능 제거
  • 개선 사항

    • 폴더 추가 요청에 대한 유효성 검사 강화

Cassiiopeia and others added 2 commits February 23, 2026 14:29
- addPlaceToFolder에 @Valid 어노테이션 추가
- Folder 엔티티 @Setter 제거 → updateName(), updateVisibility() 도메인 메서드로 변경
- UniqueConstraint + Soft Delete 충돌 해결: 재추가 시 기존 Soft Delete 레코드 restore() 복원

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- BookmarkDto: .folder(memberPlace.getFolder()) 참조 제거
- MemberPlaceService: updateFolder() 호출 블록 제거
- UpdateBookmarkRequest: folder 필드 없이 생성 (memo, rating, visited, visitedAt)
- MemberPlaceRepository: 북마크 관련 쿼리 메서드 추가
- ErrorCode: CANNOT_UPDATE_UNSAVED_PLACE 추가

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

폴더 관리 기능을 강화하기 위해 여러 계층의 코드를 재구성했습니다. BookmarkDto와 UpdateBookmarkRequest에서 폴더 필드를 제거하고, Folder 엔티티의 Lombok 세터를 명시적 업데이트 메서드로 대체했으며, FolderService의 소프트 삭제 처리 로직을 개선했습니다.

Changes

Cohort / File(s) Summary
ErrorCode 정리
MS-Common/src/main/java/.../exception/constant/ErrorCode.java
CANNOT_UPDATE_UNSAVED_PLACE와 INVALID_RATING 사이의 불필요한 공백 라인 제거
Bookmark DTO에서 폴더 필드 제거
MS-Place/src/main/java/.../place/dto/BookmarkDto.java, MS-Place/src/main/java/.../place/dto/UpdateBookmarkRequest.java
BookmarkDto와 UpdateBookmarkRequest에서 folder 필드 및 관련 @Schema 주석 제거, 북마크 업데이트 시 폴더 수정 기능 제거
Folder 엔티티 메서드 개선
MS-Place/src/main/java/.../place/entity/Folder.java
@Setter 주석 제거 후 updateName(String)과 updateVisibility(FolderVisibility) 명시적 메서드 추가
FolderPlace 저장소 확장
MS-Place/src/main/java/.../place/repository/FolderPlaceRepository.java
소프트 삭제 필터링 없이 FolderPlace를 조회하는 findByFolderAndPlace(Folder, Place) 메서드 추가
FolderService 소프트 삭제 처리
MS-Place/src/main/java/.../place/service/FolderService.java
Folder 세터 호출을 updateName/updateVisibility로 대체, addPlaceToFolder/addPlaceToDefaultFolder에서 소프트 삭제된 FolderPlace 복구 로직 추가
MemberPlaceService 폴더 업데이트 제거
MS-Place/src/main/java/.../place/service/MemberPlaceService.java
updateBookmark에서 request.getFolder() 업데이트 로직 제거
FolderController 검증 강화
MS-Web/src/main/java/.../web/controller/FolderController.java
addPlaceToFolder의 @RequestBody 파라미터에 @Valid 주석 추가

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related issues

Possibly related PRs

  • 피드 관련 API 추가 필요 #23: BookmarkDto, UpdateBookmarkRequest, MemberPlaceService의 북마크 업데이트 로직을 모두 수정하므로 변경 사항이 중복되고 충돌 가능성 있음
✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 20260223_#26_폴더_CRUD_및_폴더_장소_관리_API_구현_필요

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Cassiiopeia Cassiiopeia merged commit e6b56ed into main Feb 23, 2026
1 check was pending
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