Skip to content

feat: 채팅방 목록/상세정보 응답에 탈퇴 회원 식별 컬럼 추가#486

Merged
Yoosejeong merged 4 commits into
developfrom
feat/#481
May 4, 2026
Merged

feat: 채팅방 목록/상세정보 응답에 탈퇴 회원 식별 컬럼 추가#486
Yoosejeong merged 4 commits into
developfrom
feat/#481

Conversation

@Yoosejeong
Copy link
Copy Markdown
Collaborator

@Yoosejeong Yoosejeong commented Apr 30, 2026

🧩 관련 이슈

🛠️ 작업 내용

  • 상대가 탈퇴 회원인 채팅방을 조회하면 USER_NOT_FOUND(404)가 발생해 채팅 상세 조회가 불가능하던 문제 수정.
  • 채팅 목록, 채팅 상세조회 응답에 withdrawn 필드 추가
  • 탈퇴 회원/활성 회원 상세조회 테스트 코드 추가

📢 참고 및 특이사항

  • BlockService.isBlocked가 양쪽 user를 findActiveById로 로드하면서 차단 관계 확인뿐 아니라 사용자 활성 검증까지 책임지고 있었음.
  • 상대가 탈퇴 soft delete 상태면 활성 검증에서 404가 던져져 채팅 조회가 막혔음.
  • 그래서 차단 검사 로직 단순화를 진행함.
    • findActiveById 두 번 호출하던 부분 제거
    • existsByBlocker_IdAndBlocked_Id(Long, Long)를 추가해 ID 기반 exists 쿼리 한 번으로 차단 관계만 확인

✅ 체크리스트

  • Merge 대상 브랜치가 develop 인지 확인
  • PR 제목 형식 준수 (예: feat: 로그인 기능 구현)
  • 관련 이슈 연결 (close #이슈번호)
  • 불필요한 코드/주석 제거
  • 기능 정상 작동 확인

@Yoosejeong Yoosejeong requested a review from kkch1012 April 30, 2026 17:43
@Yoosejeong Yoosejeong self-assigned this Apr 30, 2026
@Yoosejeong Yoosejeong added the ✨ feature 새로운 기능 추가 label Apr 30, 2026
@Yoosejeong Yoosejeong linked an issue Apr 30, 2026 that may be closed by this pull request
4 tasks
@Yoosejeong Yoosejeong changed the title [FEAT] 채팅방 목록/상세정보 응답에 탈퇴 회원 식별 컬럼 추가 feat: 채팅방 목록/상세정보 응답에 탈퇴 회원 식별 컬럼 추가 Apr 30, 2026
@Yoosejeong Yoosejeong added the D-1 24시간 이내에 처리가 필요합니다. label Apr 30, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces logic to handle withdrawn users in chat rooms by masking their personal information and adding a 'withdrawn' status flag to the response DTOs. It also refactors the ChatRoomConverter to centralize DTO construction and optimizes the BlockService by utilizing ID-based repository checks to avoid unnecessary entity lookups. Review feedback suggests using descriptive placeholders for withdrawn users' nicknames instead of null to improve the client-side experience and recommends preserving the actual block status for withdrawn users rather than hardcoding it to false.

Comment thread src/main/java/com/fmi/domain/chatroom/converter/ChatRoomConverter.java Outdated
@github-actions github-actions Bot added D-0 12시간 이내에 처리가 필요합니다. and removed D-1 24시간 이내에 처리가 필요합니다. labels May 2, 2026
@Yoosejeong Yoosejeong added D-0 12시간 이내에 처리가 필요합니다. and removed D-0 12시간 이내에 처리가 필요합니다. labels May 3, 2026
@Yoosejeong Yoosejeong requested review from hyeonjaez and kkch1012 and removed request for kkch1012 May 3, 2026 13:49
@Yoosejeong Yoosejeong merged commit 3d9876d into develop May 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-0 12시간 이내에 처리가 필요합니다. ✨ feature 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 채팅방 상세정보 응답에 탈퇴 회원 식별 컬럼 추가

1 participant