Skip to content

Conversation

@ShipFriend0516
Copy link
Member

Note

작동 되는지 보고 커밋해주세요...ㅠ

  • 작성자: 서정우
  • 작성 날짜: 2024.12.03

✅ 체크리스트

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

🧩 작업 내용

  • 리액션 안보이던 문제 해결
  • 미디어 상태 공유 안되던 문제 해결

📝 작업 상세 내역

리액션

리액션 박스 앞에 z-10으로 되어있어서 가려서 안보였던 문제 해결

<div className="absolute inset-0 bg-gray-black rounded-custom-l overflow-hidden **z-10**">
        <DisplayMediaStream mediaStream={stream} isLocal={isLocal} />

미디어 상태 공유 안되던 문제 해결

위에 onopen에서 채널을 ref 객체에 이미 넣어주고 있는데 channel.onopen 해서 다시 넣는 코드 때문에 연결이 안된 데이터채널이 저장되고 있었습니다.
ondatachannel은 상대방과 데이터채널이 수립될 때 실행되는거라 저장하는 채널과는 다른 채널입니다.

channel.onopen 부분 지우니 정상작동했습니다.

image

🐥 리뷰 받고 싶은 포인트(선택)

  • 제가 안쓴 코드에서 버그 찾으려니 힘드네요.. webrtc라 디버깅이 힘든 것 같습니다

@ShipFriend0516 ShipFriend0516 added the 🐛 Bug Fix 해충 제거 label Dec 3, 2024
@ShipFriend0516 ShipFriend0516 self-assigned this Dec 3, 2024
Copy link
Member

@yiseungyun yiseungyun left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 👍

@blu3fishez blu3fishez merged commit 965e53b into boostcampwm-2024:dev Dec 3, 2024
@ShipFriend0516 ShipFriend0516 deleted the fix/media-bug branch December 5, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug Fix 해충 제거

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants