Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API: ✏️ 채팅방 정보 조회 응답 수정 #191

Merged
merged 13 commits into from
Nov 5, 2024

Conversation

psychology50
Copy link
Member

작업 이유

  • iOS 측 요구 사항에 따른 API 스펙 수정

작업 사항

  1. 채팅방 정보 조회 시, otherParticipants 필드에 name 필드 추가
  2. 최근 활동 이력에 admin이 없거나, 조회하는 클라이언트가 admin이 아니라면 조회 쿼리 추가.
  3. 채팅방 멤버 상세 정보에서 userId 필드 추가.

리뷰어가 중점적으로 확인해야 하는 부분

  • 머리가 안 돌아가서, 일단 동작하는 코드로 구현했는데 덕분에 코드가 심각하게 더럽습니다.

발견한 이슈

  • 채팅방 메시지 이력이 조회가 안 되는데, 소켓 서버 이미지가 최신화 반영이 안 된 듯. 일단 로컬로 이미지 올리겠습니다.

@psychology50 psychology50 added the fix 기능 수정 label Nov 5, 2024
@psychology50 psychology50 self-assigned this Nov 5, 2024
@psychology50 psychology50 merged commit 727c2df into dev Nov 5, 2024
1 check passed
@psychology50 psychology50 deleted the fix/more-details-chat-room-data branch November 5, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant