[Feat] 세션 입장 로직 구현 및 세션 내부 이벤트에 대한 대응 구현 #121
Merged
Conversation
This file contains hidden or 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
- RedisService : 레디스 클라이언트가 작업할 일련의 공통 처리 작업 모음
- MemberConnection : 각 방에 참여하는 유저의 연결 상세 정보 - Room : 방의 상세정보(메타데이터)
- create-room.dto.ts : 방 생성 시 필요한 객체 정보 정의
- 스터디 세션 생성과 관련된 비즈니스 로직 - `Redis` 등의 저장소 정보와 관심분리 목적
- `Redis subkey` 등 서브키들을 구축하는데 있어서 해당 모듈 사용 - `Redis` 자체 클라이언트와 관심 분리, 동시에 비즈니스 로직과 연관된 작업들 명시
- 방 생성에 대한 이벤트 핸들러 추가 - 방 참가에 대한 로직 변경 중..
- 레포지토리 추가
- primitive 타입인지 object 타입인지는 개발자에게 검사하기로 넘김 - 로그문 삭제
- Postman 측에서 생긴 에러였으므로 로그문 삭제 - 추후 `DTO`와 `Pipe` 이용하여 validation 로직 추가 예정
- members 항목은 사용하지 않으므로 삭제 - 사용하지 않는 주석, 로그문 제거
…ession feature/study session
- 여러 서비스의 작업으로 이루어지도록 수정 - 서비스의 작업이 간단해지도록 수정 - 방을 나가기 / 방장을 변경하기 작업을 나눔
- null 일 경우 false 를 반환하도록 `checkHost` 수정 - getRoomById 를 통해 받아오는 데이터를 단언하여 `Room` 으로 타입화 - 타입 가드를 사용할 예정 - host를 설정해서 받아오는 로직 수정 - 이는 추후 서비스로 올릴 예정
- 최대 1~5인 설정 가능
- 머지 과정에서 사라진 코드 복구
- `GET /api/rooms` 시, status != PRIVATE 인 모든 방 Record 반환
- 단순 소켓 배열에서 방장 정보를 확인할 수 있도록 변경했습니다.
- `all_users` 에 방 정보 추가 - `room_created` 에 방 정보 추가
…/study-session [Feat] 스터디 세션 기능 서버측 기능 추가
- 토스트 메시지로 유저 퇴장 안내
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Note
작성자: 서정우
작성 날짜: 2024.11.15
관련 에픽: #3
✅ 체크리스트
🧩 작업 내용
📝 작업 상세 내역
세션 메타데이터
방장 전용 기능
세션 종료시