Skip to content

🎨feat : Socket module 정리 및 manager control 추가 - #3

Open
raymondanythings wants to merge 7 commits into
masterfrom
feature/#3-socket-optimization
Open

🎨feat : Socket module 정리 및 manager control 추가#3
raymondanythings wants to merge 7 commits into
masterfrom
feature/#3-socket-optimization

Conversation

@raymondanythings

Copy link
Copy Markdown
Member

✅ PR 체크리스트

  • 코드 스타일 가이드 규칙을 준수했어요.

PR 타입

설명하실 변경 사항은 어떤 종류인가요?

  • 버그 수정
  • 기능 추가
  • 코드 스타일 업데이트
  • 기타

이슈 번호 : #3

해당 PR이 프로젝트에 큰 영향이 있나요?

  • Yes
  • No

작업 내용

  • socket manager를 활용하여 namespace 분기처리

추가 설명

유저 데이터를 저장하기 위해 redis scheme 생각해야함.

- Socket 신규 매소드 타입 추론 안되던 문제 해결
- Manager를 클래스화 하여 맴버변수로 기 연결된 소켓 상태 관리
- socket 이벤트 리스너 생성 시 기존 리스너가 존재할 경우 체크
Comment threadsrc/screen/Home.tsx Outdated
raymondanythingsand others added 3 commits February 5, 2023 04:07
- useSocket Error 핸들러 추가
- login, home screen -> gql query 추가
- user store 활용, 유저객체 핸들링
import store from 'store/index'
import { GetMeQuery } from '__api__/types'

const GET_ME = gql`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

graphQl 를 도입 하셨네요.

Sign up for freeto 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

@raymondanythings@HerrineKim@cheonkyu0@ydw1996