[Feat] 공감 기능 구현, 렌더링 최적화와 깜빡임 이슈 해결 #108
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
…도록 수정 - handleThumbsUp => handleReaction
- senderId, reaction 을 보내서 클라이언트에서 처리할 수 있도록 구현
- 시행착오를 많이 하느라 커밋을 나누지 못했다. - 공감 기능 구현은 금방했는데 깜빡임 문제해결을 하느라 오래걸렸다. - ref를 전달하는 방식에서 stream을 전달하는 방식으로 수정 - stream 송출만을 담당하는 DisplayMediaStream 컴포넌트 분리
ShipFriend0516
changed the title
Closed
3 tasks
yiseungyun
approved these changes
Nov 11, 2024
Member
yiseungyun
left a comment
yiseungyun
left a comment
There was a problem hiding this comment.
화면 깜빡임 트러블슈팅 흥미롭네요. 👍 고수.
수고많으셨습니다. ☃️
blu3fishez
approved these changes
Nov 12, 2024
twalla26
approved these changes
Nov 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.11
관련 에픽: #11
공감 기능을 구현했습니다. 수민님이 시그널링 서버 코드를 잘 만들어주셔서 참고해서 시그널링 서버를 중개해서 공감 이벤트를 수신하고 세션에 재발신하도록 하는 방식으로 구현했습니다.
공감 기능 자체는 금방 구현했으나 클라이언트에서 비디오컴포넌트 깜빡임 이슈가 있어서 렌더링 최적화를 하느라 시간이 생각보다 지체되었네요..
그래도 성과로는 비디오 컴포넌트 렌더링 최적화와 스트림 송출 책임 분리를 얻었습니다.
✅ 체크리스트
🧩 작업 내용
📝 작업 상세 내역
공감 기능
reaction이벤트에 body에 roomId와 reactionType을 보내도록 구현비디오 컴포넌트 최적화
useMemo훅을 사용하여 peers의 변동이 있을때만 리렌더링 하도록 최적화VideoContainer컴포넌트에서 ref를 전달받던 것을 stream을 전달 받아 하위에서 ref를 사용하도록 구조 변경VideoContainer컴포넌트 내부 video 태그와 스트림 송출의 역할을DisplayMediaStream컴포넌트로 분리깜빡임 이슈 해결
📌 테스트 및 검증 결과
💬 다음 작업 또는 논의 사항
📎 참고 자료 (선택)
🐥 리뷰 받고 싶은 포인트(선택)