Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cam] WebRTC 쪽 코드 리팩토링 및 적절히 훅 분리 필요 #3

Open
wjdwl002 opened this issue Feb 12, 2023 · 1 comment
Open
Assignees
Labels
refactoring works quite well but needs a refactoring

Comments

@wjdwl002
Copy link
Member

src>components>Cam 하위의 컴포넌트들 전반적으로 코드 리팩토링 및 적절히 훅을 분리할 필요가 있음.

리액트 컴포넌트에서 socket 객체를 다루는 다른 클론코딩 사례를 검토하고 적절한 방식을 도입할것

컴포넌트 용도 설명

CamList.tsx

여러 캠들을 가지고 있는 캠 컨테이너 컴포넌트로, 본인이 입장하는것과 다른 멤버가 입장하는 것들에 대한 처리를 이 컴포넌트에서 하고있음

LocalCam.tsx

내 캠 화면 컴포넌트로, mute와 cam on/off 관리를 함

RemoteCam.tsx

상대방 캠 화면 컴포넌트로, 재사용 가능하게(명수만큼 n번 이상 호출되게) 설계되었으며, 구글의 stun 서버를 이용해 peerConnection 하는 코드가 이 컴포넌트에서 관리되고있음.

SignalingConenct.tsx

초기 진입시 캠 연결을 위한 signaling server 와의 sdp 교류 및 ice candidate 선정

@wjdwl002 wjdwl002 added the refactoring works quite well but needs a refactoring label Feb 12, 2023
@wjdwl002
Copy link
Member Author

wjdwl002 commented Feb 12, 2023

@Charmull 지민님 혹시 컴포넌트 설명에 덧붙힐 말이 있으면 수정 바랍니다 😇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring works quite well but needs a refactoring
Projects
None yet
Development

No branches or pull requests

2 participants