Skip to content

Conversation

@loseminho
Copy link
Collaborator

📌 개요

1. 기존에는 Redis에 등록을 하기 위해서는 REST API, WebSocket 사용하는 두가지 경로로 잡혀있었음.

  • REST API -> RoomRedisService 사용
  • WebSocket -> RoomParticipanService 사용

결국엔 Redis로 모이게 되는데 두가지 경로를 가지고 있어
중간 경로를 동일한 서비스를 사용 할 수 있게 만들기 위해서
RoomRedisService의 기능을 제거시키고, RoomParticipanService 경로로 통합 시켰습니다.

2. 기존에 역할을 변경시킬 시, DB 저장만 저장

🔨 작업 내용

RoomRedisService는 전달의 역할만 진행하고 웹소켓세션매니저가 일을 하고 있어 RoomParticipanService로 통합

  • Before: RoomService → RoomRedisService → SessionManager → ParticipantService
  • After: RoomService → ParticipantService

역할을 변경시킬 시, DB 저장 + 웹 소켓으로 알림 전송 -> ui로 확인가능, 채팅창으로 알림 할 수 있음 (실시간 성 추가)

🔗 관련 이슈

Closes #169

📝 참고 사항

  • RoomRedisService을 바로 삭제하는 것이 아니라 임시로 Deprecated 했습니다.

✅ 체크리스트

  • 기능 동작 확인
  • 테스트 코드 작성
  • 문서/주석 추가 및 최신화

@loseminho loseminho self-assigned this Oct 4, 2025
@github-actions github-actions bot changed the title Refactor : 스터디룸 웹소켓 연동과 역할 변경 알림 Refactor : 스터디룸 웹소켓 연동과 역할 변경 알림 (#146) Oct 4, 2025
@loseminho loseminho merged commit f9423f4 into dev Oct 4, 2025
2 checks passed
@github-actions github-actions bot deleted the refactor/146 branch October 4, 2025 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants