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

chat api 변경 내역 반영 및 비로그인 시 프로필 페이지 접근 불가 버그 수정 #284

Merged
merged 6 commits into from
Nov 22, 2023

Conversation

1g2g
Copy link
Member

@1g2g 1g2g commented Nov 22, 2023

⚙️ PR 타입

  • Feature
  • Hotfix

✨ 기능 설명 or 🚨 문제 상황

  • 1:1 채팅방에서 나가기를 하면 대화를 했던 상대방 이름을 가져올 수 없는 문제가 발생해 채팅 목록 페이지에 접근할 때 에러가 발생했습니다.
    상대방 이름을 가져올 수 있도록 채팅방을 구성하는 멤버 이름은 유지하되, isSenderActive로 상대방이 채팅방에 실제로 속해있는지 판별하기로 했습니다.
  • 비로그인 시 profile 페이지에 접근할 수 없는 문제도 함께 해결했습니다(대화걸기 버튼 이벤트를 훅으로 분리하면서 쿼리 수정).

👨‍💻 구현 내용 or 👍 해결 내용

fix: GetPersonalChatRoomExistedResponse 변경 내역 반영

feat: 채팅방 입장 api 추가

feat: 대화걸기 버튼 클릭 훅 작성

feat: 프로필 페이지 대화걸기 버튼 훅 적용

feat: 채팅 목록 페이지 로그인 validation 작성

🎯 PR 포인트

📝 참고 사항

게스트 모집 상세 페이지에서 아래 코드를 가져다 사용하시면 좋을 것 같습니다!

  const { handleClickChattingButton } = useChatOnButtonClick({
    targetId: match.host.id,
    targetNickname: match.host.nickname,
    navigate,
    myId: loginInfo?.id ?? null,
  });

❓ 궁금한 점

@1g2g 1g2g added 긴급 긴급한 수정사항으로 바로 리뷰해 주세요. 버그 코드의 버그를 수정합니다. labels Nov 22, 2023
@1g2g 1g2g added this to the 5차 스프린트 milestone Nov 22, 2023
@1g2g 1g2g self-assigned this Nov 22, 2023
@1g2g 1g2g linked an issue Nov 22, 2023 that may be closed by this pull request
@1eecan
Copy link
Contributor

1eecan commented Nov 22, 2023

🎉 @dlwl98 님 랜덤 리뷰어로 당첨되셨습니다! 리뷰를 부탁드립니다. 🙏

@1eecan 1eecan merged commit 3fcf23d into dev Nov 22, 2023
1 check passed
@1eecan 1eecan deleted the hotfix/#281-chat-api-res-changed branch November 22, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
긴급 긴급한 수정사항으로 바로 리뷰해 주세요. 버그 코드의 버그를 수정합니다.
Projects
None yet
2 participants