onConnection 핸들러에 sync-is-running 이벤트 전파를 추가 했습니다.
프론트엔드에서 중간에 관전하는 유저의 경우 isRunning 값이 연동 안되는 문제를 해결하기 위해 기능을 추가 했습니다.
PR Point
onRoomDetailConnection 에서 DB 정보를 가져오는 부분이 어떤건 helper 함수, 어떤건 try/catch 내에서 Service 메서드를 사용하는 식으로 되어 있어서 추후 리펙터링 할 때에는 공통 에러 핸들러를 소켓에 적용시키고 Service 메서드만 사용하는 식으로 변경하면 좋겠다는 생각이 들었습니다.
📸 스크린샷
방에 들어온 직후 소켓을 통해 sync-is-running 이벤트를 전파 받은걸 확인할 수 있습니다.
lucaseunchae
commented
4 months ago
Pull Request
작업한 내용
onConnection핸들러에sync-is-running이벤트 전파를 추가 했습니다.PR Point
onRoomDetailConnection에서 DB 정보를 가져오는 부분이 어떤건 helper 함수, 어떤건 try/catch 내에서 Service 메서드를 사용하는 식으로 되어 있어서 추후 리펙터링 할 때에는 공통 에러 핸들러를 소켓에 적용시키고 Service 메서드만 사용하는 식으로 변경하면 좋겠다는 생각이 들었습니다.📸 스크린샷
sync-is-running이벤트를 전파 받은걸 확인할 수 있습니다.closed #87