fix: 새로고침 할때 가끔 채팅구독이 안되는 문제 해결 #142
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
연결될때 useState를 변경해서 구독을 트리거 함
cheonjiyun
added a commit
that referenced
this pull request
Dec 22, 2024
* feat : 서버 api 요청 주소 변경 * chore : dev cicd 파이프라인 작성 * chore : dev CI/CD 파이프라인 수정 * chore : prod CI/CD 작성 * refactor: 채팅을 웹소켓으로 교체한다 (#138) * fix: 새로고침하면 본인 직업이 날라가는 문제 해결 (#140) * fix: 새로고침 할때 가끔 채팅구독이 안되는 문제 해결 (#142) 연결될때 useState를 변경해서 구독을 트리거 함 * feat: 대기방 정보 SSE로 교체 (#144) * feat: 대기방 정보 SSE로 교체 * chore: 사용안하는 코드 및 log제거 * feat: 마피아 스킬을 웹소켓으로 교체 (#146) * Feature/148 채팅에서 직업 보여주기 (#150) * chore: 글씨 제거 * feat: 채팅에서 직업 찾아서 보여주기 * Feature/#147 깃허브 액션에 env 추가 (#151) * feat: 초대코드 도메인 두번 들어가는 형상 제거 * feat: 웹소켓 baseURL에 도메인 변수를 활용하도록 수정 * feat: 도메인 상수를 env를 활용하도록 수정 * feat: 깃허브 액션에 env 추가 * fix: 마피아일 때 기존 타켓 요청이 안가는 버그 수정 --------- Co-authored-by: waterricecake <[email protected]> Co-authored-by: waterricecake <[email protected]>
cheonjiyun
added a commit
that referenced
this pull request
Dec 26, 2024
* feat : 서버 api 요청 주소 변경 * chore : dev cicd 파이프라인 작성 * chore : dev CI/CD 파이프라인 수정 * chore : prod CI/CD 작성 * refactor: 채팅을 웹소켓으로 교체한다 (#138) * fix: 새로고침하면 본인 직업이 날라가는 문제 해결 (#140) * fix: 새로고침 할때 가끔 채팅구독이 안되는 문제 해결 (#142) 연결될때 useState를 변경해서 구독을 트리거 함 * feat: 대기방 정보 SSE로 교체 (#144) * feat: 대기방 정보 SSE로 교체 * chore: 사용안하는 코드 및 log제거 * feat: 마피아 스킬을 웹소켓으로 교체 (#146) * Feature/148 채팅에서 직업 보여주기 (#150) * chore: 글씨 제거 * feat: 채팅에서 직업 찾아서 보여주기 * Feature/#147 깃허브 액션에 env 추가 (#151) * feat: 초대코드 도메인 두번 들어가는 형상 제거 * feat: 웹소켓 baseURL에 도메인 변수를 활용하도록 수정 * feat: 도메인 상수를 env를 활용하도록 수정 * feat: 깃허브 액션에 env 추가 * fix: 마피아일 때 기존 타켓 요청이 안가는 버그 수정 * refactor: SSE와 웹소켓 URL을 한 파일에서 관리한다. * refactor: 초대코드 저장 및 밤 마피아 url 변경 * fix: wait상태일때 내 직업 삭제 * fix: 경찰 이미 사용했을 때는 토스트, 아직 안왔을 때는 안띄움 --------- Co-authored-by: waterricecake <[email protected]> Co-authored-by: waterricecake <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 내용
새로고침할 때 생기는 문제가 아니였다. 랜덤하게 발생하는 문제였다.
소켓 연결이 끊기는게 아니라, 가끔 소켓 연결이 되기전에 구독함수를 먼저 하려고 한다.
(비동기 함수도 아닌데 왜 실행순서를 안지키는지 모르겠다;;)
useState를 하나 추가-> useEffect 의존성 배열에 추가 -> 웹소켓 연결이 될 때 상태 변경 -> 트리거 되어서 구독하는 함수를 한번더 호출
따라서 먼저 구독을 시도했을 경우, 연결됐을 때 한 번 더 실행함으로서 해결.
close #141