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

유저를 관리하기 위한 웹소켓 Room 추가 #210

Merged
merged 6 commits into from
Nov 19, 2024

Conversation

yewonJin
Copy link
Collaborator

🔖 연관된 이슈

📂 작업 내용

  • Client 룸 생성하기
  • 유저 식별을 위한 랜덤 색상 및 clientId 생성 (로컬스토리지 이용)
  • 에디터 접속 인원을 노드와 에디터에 보여주기

📑 참고 자료 & 스크린샷 (선택)

Recording.19.mp4

📢 리뷰 요구사항 (선택)

일단 디자인은 최대한 간단하게 했습니다. (추후 수정을 위해)
궁금한 게 하나 있는데, 자신의 접속 현황은 표시를 안하는 게 자연스러울까요??

추후 수정해야 할 사항

  1. 에디터(content)에서 클라이언트를 일관되게 보여주기 (에디터 마다 다른 웹 소켓 Room에 접속해서)
  2. 중복 탭 문제 -> 클라이언트 id가 같은 접속 현황이 여러 개 보여지게 됨
  3. 이미 접속중인 clientId 가 있으면 이전 연결을 끊고, 새로운 연결을 해야 할 거 같습니다! (새로고침하면 flow-room에 다시 연결돼서 컴포넌트 key 문제가 생깁니다.

@yewonJin yewonJin added 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우 labels Nov 19, 2024
@yewonJin yewonJin requested a review from djk01281 November 19, 2024 12:16
@yewonJin yewonJin self-assigned this Nov 19, 2024
@yewonJin yewonJin linked an issue Nov 19, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@djk01281 djk01281 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 👏🏻👏🏻👏🏻 사용자는 모두 표시되어도 괜찮을 것 같습니다..!

@djk01281 djk01281 merged commit 2ddd6ac into develop Nov 19, 2024
9 checks passed
@Tolerblanc Tolerblanc deleted the feature-fe-#206 branch November 20, 2024 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

유저를 관리하기 위한 웹소켓 Room 추가
2 participants