Skip to content

Conversation

@ccjngwn
Copy link
Contributor

@ccjngwn ccjngwn commented Dec 30, 2025

#️⃣연관된 이슈

#29

📝작업 내용

  • Friendship(친구 관계) 도메인 구현(추가, 삭제, 수정, 조회)

  • ChatTopic(채팅 주제) 도메인 구현(조회)

  • 친구 관계 API 보안(소유자 검증 로직)

  • Swagger 문서화

  • Swagger 의존성 변경(2.7.0)

🛠️주요 변경 사항

  • 기능 추가
  • 버그 수정
  • 문서 업데이트
  • 코드 리팩토링
  • 테스트 추가 또는 수정
  • 의존성 추가/삭제

📸스크린샷

💬리뷰 요구사항

  • FriendshipId를 PathVariable로 전달받는데 보안상 괜찮은 지 궁금합니다.
  • 친구 추가 단계가 친구 관계 생성 + 친구 관계 설정으로 2단계로 나누어져 있어 speechStyle과 chatGoal의 null 값을 허용했는데 더 좋은 방식이 있을지 피드백 부탁드립니다.
  • 전체적으로 문서화 추가가 필요한 부분이 있다면 피드백 부탁드립니다.
  • 처리하지 못한 오류가 발생할만한 플로우가 있는지 확인 부탁드립니다.

@ccjngwn ccjngwn self-assigned this Dec 30, 2025
@ccjngwn ccjngwn added the 정원 label Dec 30, 2025
@ccjngwn ccjngwn merged commit 15ad7e4 into develop Jan 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants