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.
📝작업 내용
스크린샷 (선택)
💬리뷰 요구사항(선택)
VideoCall이라고 이름 붙였는데 보다보니까 별로인 것 같습니다. webRTC도 별로고 webShare가 적당해 보입니다. 그런데 화면 공유 + 음성 기능까지 할 것 같아서 VideoCall이라고 했는데 괜찮을까요?
화면공유 하려면 게더타운에 실시간으로 누가 들어왔는지 알아야 합니다. > 이 기능이 시그널링 서버에 있습니다. 지현님도 이 소켓이랑 이 기능을 사용하셔야 할 것으로 추측되는데
그런데 이게 제가 맡아서 작성을 해도 되는지 잘 모르겠습니다. 역할 분담을 어떻게 하는 게 좋을까요?(한 파일을 같이 수정하니까 유지보수가 어려울 수도 있을 것 같아서)
-> 2번 결론: 소켓 관련 부분은 제가 쭉 담당하는 게 학습 곡선 및 유지보수에 좋을 것 같습니다.
제가 api 계층을 하나 더 만들어서 소켓 관련 api를 따로 파일로 분리한 다음에, 그냥 import하셔서 사용할 수 있도록 하는 게 좋아보인다!
<<중요>>소켓 관련 의존성 설치가 필요합니다!!!
$ yarn add express socket.io
$ yarn add socket.io socket.io-client
만약 yarn start하셨을 때 오류가 난다면 이 둘 명령어가 필요합니다