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.

Sign up for GitHub

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

Jump to bottom

[Refactor/207] 채팅 API에 탈퇴 관련 로직 추가 #208

Merged
merged 10 commits into from
Sep 8, 2024
Merged

[Refactor/207] 채팅 API에 탈퇴 관련 로직 추가 #208

merged 10 commits into from
Sep 8, 2024

Conversation

Eunjin3395
Copy link
Member

🚀 개요

채팅 API 응답에 탈퇴 관련 값 추가
탈퇴 관련 에러처리 추가

🔍 변경사항

  • uuid 채팅방 입장, memberId 채팅방 시작 API 응답에 blind(상대 회원 탈퇴 여부) 추가
  • 채팅방 목록 조회 API 응답에 blind(상대 회원 탈퇴 여부) 추가
  • memberId 채팅방 시작 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • uuid 채팅방 입장 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • 채팅방 목록 조회 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • 탈퇴, 상대방이 차단, 내가 차단, 기존 채팅방 유무, 입장 유무에 따른 모든 case의 에러 처리
  • 검증 로직 모듈화
  • ErrorStatus enum 이름 변경

⏳ 작업 내용

  • uuid 채팅방 입장, memberId 채팅방 시작 API 응답에 blind(상대 회원 탈퇴 여부) 추가
  • 채팅방 목록 조회 API 응답에 blind(상대 회원 탈퇴 여부) 추가
  • memberId 채팅방 시작 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • uuid 채팅방 입장 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • 채팅방 목록 조회 API 응답의 gameName 추출 로직을 탈퇴한 회원도 고려하도록 변경
  • 탈퇴, 상대방이 차단, 내가 차단, 기존 채팅방 유무, 입장 유무에 따른 모든 case의 에러 처리
  • 검증 로직 모듈화
  • ErrorStatus enum 이름 변경

📝 논의사항

@Eunjin3395
♻️ [Refactor] 채팅방 목록 조회 API 응답에 blind 추가
9d9e47f
@Eunjin3395
♻️ [Refactor] ChatroomEnterDTO에 blind 추가
081d6e3
@Eunjin3395
♻️ [Refactor] 새 채팅방 생성 시 chatMessageList 초기화
69746f9
@Eunjin3395
♻️ [Refactor] memberId를 통한 채팅방 시작 메소드 리팩토링 및 상대방 탈퇴 여부 고려
7497415
@Eunjin3395
♻️ [Refactor] boardId를 통한 채팅방 시작 메소드 리팩토링 및 상대방 탈퇴 여부 고려
862312e
@Eunjin3395
♻️ [Refactor] 매칭을 통한 채팅방 시작 메소드 리팩토링 및 상대방 탈퇴 여부 고려
4a5d7c0
@Eunjin3395
♻️ [Refactor] readChatMessages 메소드 리팩토링
c1a82d6
@Eunjin3395
♻️ [Refactor] addChat 메소드 리팩토링
2719793
@Eunjin3395
♻️ [Refactor] 채팅방 목록 조회 API의 gameName 탈퇴 여부 고려
2a49c9b
@Eunjin3395
Merge branch 'develop' into refactor/207
ecdb706
@Eunjin3395 Eunjin3395 self-assigned this Sep 8, 2024
@Eunjin3395 Eunjin3395 linked an issue Sep 8, 2024 that may be closed by this pull request
♻️ [Refactor] 채팅 API 응답에 탈퇴 관련 값 추가 #207
Closed
8 tasks
@Eunjin3395 Eunjin3395 merged commit aa5c454 into develop Sep 8, 2024
1 check failed
@Eunjin3395 Eunjin3395 deleted the refactor/207 branch September 8, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rimi3226 rimi3226 Awaiting requested review from rimi3226

@hzee97 hzee97 Awaiting requested review from hzee97

Assignees

@Eunjin3395 Eunjin3395

Labels
♻️ refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

♻️ [Refactor] 채팅 API 응답에 탈퇴 관련 값 추가
1 participant
@Eunjin3395