Skip to content

Conversation

@twalla26
Copy link
Collaborator

@twalla26 twalla26 commented Nov 14, 2024

관련 이슈 번호

  • 작성자: 송수민
  • 작성 날짜: 2024.11.14

✅ 체크리스트

  • 코드가 정상적으로 작동하는지 확인했습니다.
  • 주요 변경사항에 대한 설명을 작성했습니다.
  • 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

  • 백엔드 팀원이 구현한 각 코드들을 merge하고 공유하기 위한 PR입니다!
  • 최종 코드는 테스트를 거친 후에 dev 브랜치로 PR을 올릴 예정입니다.

blu3fishez and others added 13 commits November 13, 2024 22:42
- RedisService : 레디스 클라이언트가 작업할 일련의 공통 처리 작업 모음
- MemberConnection : 각 방에 참여하는 유저의 연결 상세 정보
- Room : 방의 상세정보(메타데이터)
- create-room.dto.ts : 방 생성 시 필요한 객체 정보 정의
- 스터디 세션 생성과 관련된 비즈니스 로직
- `Redis` 등의 저장소 정보와 관심분리 목적
- `Redis subkey` 등 서브키들을 구축하는데 있어서 해당 모듈 사용
- `Redis` 자체 클라이언트와 관심 분리, 동시에 비즈니스 로직과 연관된 작업들 명시
- 방 생성에 대한 이벤트 핸들러 추가
- 방 참가에 대한 로직 변경 중..
- primitive 타입인지 object 타입인지는 개발자에게 검사하기로 넘김
- 로그문 삭제
- Postman 측에서 생긴 에러였으므로 로그문 삭제
- 추후 `DTO`와 `Pipe` 이용하여 validation 로직 추가 예정
- members 항목은 사용하지 않으므로 삭제
- 사용하지 않는 주석, 로그문 제거
@twalla26 twalla26 merged commit 14d0b43 into boostcampwm-2024:feature/study-session Nov 14, 2024
@ShipFriend0516
Copy link
Member

@twalla26 @blu3piece 이거 PR dev 브랜치가 아닌데 그냥 머지해서 써야하나요?

@twalla26
Copy link
Collaborator Author

@twalla26 @blu3piece 이거 PR dev 브랜치가 아닌데 그냥 머지해서 써야하나요?

이 코드는 백엔드 간의 협업을 위한 브랜치에 올린 코드이고, 곧 찬우님께서 최종 코드를 dev 브랜치에 올려주실 예정입니다!

@twalla26 twalla26 changed the title feature/study session [merge] 백엔드 로직 merge 및 conflict 해결 Nov 14, 2024
@twalla26 twalla26 changed the title [merge] 백엔드 로직 merge 및 conflict 해결 [merge] 백엔드 코드 merge 및 conflict 해결 Nov 14, 2024
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.

3 participants