Skip to content

Conversation

@yiseungyun
Copy link
Member

관련 이슈 번호

  • 작성자: 이승윤
  • 작성 날짜: 2024.11.14

✅ 체크리스트

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

🧩 작업 내용

  • 백엔드 서버와 연결해서 방 생성 확인
  • 만들어진 방 링크로 다른 사용자 접속
  • 백엔드 데이터 형식으로 인한 코드 수정

📝 작업 상세 내역

세션 생성 후 입장

스크린샷 2024-11-14 오후 7 59 27
  • 세션 만들기에서 create_room으로 서버에 요청하면 서버가 roomId를 주고, 해당 roomId를 이용해 접속한다.
  • 상대방도 session/:sessionId를 같이 입력하고 닉네임을 입력하면 같은 방에 접속할 수 있다.
  • 기존의 roomId 입력 부분이 이제 필요없어져서 지웠다.
  • 입장하면 사용자가 입력한 이름으로 뜨는데, 링크로 들어온 상대는 방 이름을 볼 수가 없다.
    • 추후 서버에서 만들어진 방 title도 받아와서 넣는게 필요해보인다.
  • 백엔드에서 배열로 소켓 아이디만을 전달해서 화상회의 기능이 동작하지 않았으나, 데이터 형식을 수정해서 그 부분에 맞게 수정한 후 오류가 해결되었다.

💬 다음 작업 또는 논의 사항

  • 급하게 작업하느라 코드를 한 번 더 확인할 필요가 있을거 같음

- response.success로 조건 검사가 불가능해 resposne.roomId가 있으면 해당 아이디로 입장하게 변경
- 기존에 roomId 입력 부분 삭제 -> 닉네임 입력 후 바로 해당 방으로 입장 가능하게 구현
- 링크를 받은 사용자도 닉네임 입력 후 같은 방으로 들어갈 수 있게 구현
- 백엔드 쪽에서 보내는 데이터 형식 변경으로 인한 데이터 처리 부분 수정
- 이벤트 리스너들 함수로 정의해서 관리
Copy link
Member

@ShipFriend0516 ShipFriend0516 left a comment

Choose a reason for hiding this comment

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

고생하셨습니당

Copy link
Member

Choose a reason for hiding this comment

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

이거 어떻게하는지 하고 싶었는데 감사합니다

Copy link
Member

Choose a reason for hiding this comment

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

세션의 아이디로 이름을 가져오는 이벤트나 아니면 join_room 할 때 받아오도록 해야겠네요 createRoom 할때도 세션 이름을 제공해야할 것 같아보입니다 굳굳

question: string;
participants: string[];
roomId: string;
roomId: string | undefined; // TODO: sessionId가 입력되지 않았을 때(undefined) 처리 필요
Copy link
Member

Choose a reason for hiding this comment

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

확인!

@ShipFriend0516 ShipFriend0516 merged commit c0bd0b2 into dev-fe Nov 14, 2024
@yiseungyun yiseungyun removed this from the 스터디 세션 기능 milestone Nov 19, 2024
@ShipFriend0516 ShipFriend0516 deleted the feature/create-room branch November 19, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants