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

[feat] 채팅방 생성하기 API 연동 #218

Closed
wants to merge 21 commits into from
Closed

[feat] 채팅방 생성하기 API 연동 #218

wants to merge 21 commits into from

Conversation

juyeong-s
Copy link
Contributor

💡 이슈

resolve #160

🤩 개요

채팅방 생성하기 API 연동하기 작업입니다

🧑‍💻 작업 사항

  • 이전에 소켓 연동을 했었는데 잘못된 것 같아서 다 지우고 API 함수만 다시 만들었습니다~
    • 소켓은 어짜피 사용해야하니 설치한 거 그대로 두겠습니다
  • 채팅 쪽 뭉개지거나 어색한 css 수정했습니다

📖 참고 사항

아직 서버가 안돼서 테스트는 못했습니다 완료되면 머지하겠습니다~

@juyeong-s juyeong-s added the 📡 API 서버 API 통신 작업(FE, BE 둘다) label Oct 29, 2022
@juyeong-s juyeong-s self-assigned this Oct 29, 2022
Copy link
Contributor

@kingyong9169 kingyong9169 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

toastSuccess({ message });
},
onError: (error) => {
if (isAxiosError<res.AuthError>(error) && !!error.response) {
Copy link
Contributor

Choose a reason for hiding this comment

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

요거 아마 res.Error로 바뀌었을 겁니다!~

return useCoreMutation(createChatRoom, {
onSuccess: (data) => {
const message =
'미팅 신청을 수락했습니다. 채팅방에서 자유롭게 대화해보세요!';
Copy link
Contributor

Choose a reason for hiding this comment

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

채팅방으로 이동하는 navigate만 추가하심 될 것 같아요~

@juyeong-s juyeong-s closed this Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📡 API 서버 API 통신 작업(FE, BE 둘다)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 채팅방 입장하기 기능 구현
2 participants