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

Feature/#71 로그인한 사용자는 권한이 있는 워크스페이스에 다른 사용자를 초대 구현 #250

Conversation

hyonun321
Copy link
Collaborator

@hyonun321 hyonun321 commented Nov 30, 2024

📝 변경 사항

다른 유저 워크스페이스 초대 구현

11235c2.mp4
  • 초대 수신시 해당 유저에게 토스트 토글 표시

실시간으로 접속 사용자 수 표시

11235c21.mp4
  • 접속할때마다 현재 workspace 토스트 토글로 표시
  • 현재 접속중인 workspace 모달창에서 표시(파란색)

🔍 변경 사항 설명

  • 서버에 workspace 별 접속 사용자수 정보를 저장합니다.
  • client는 switch, 소켓 재연결 이 될때마다 정보를 서버에 던집니다.
  • workspace/list를 수신할때마다 workspaceSelectModal을 재 렌더링 하여 값이 실시간으로 바뀝니다.

🙏 질문 사항

  • strictMode를 키고하면 아마 1클라이언트 당 2명이 접속할 것입니다! 스트릭트 모드를 끄고 보시면 정확한 접속 유저수를 보실수 있습니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

- 서버쪽의 workspace/list 를 초대받은 유저에게 재송신
- 유저의 아이디를 toast에 표시하게 수정

#71
…되도록 설정

- 현재 init으로 null로 초기 정보를 설정하기때문에 새로고침때 서버에서 받아오게 하려면 loading을 넣어야함. 이를 방지하기 위해 추가
- client에서 접속자수, 최대 인원수 표시
- 서버에서 접속자수를 workspace/connection 으로 api로 송신

#71
- let -> const로
@hyonun321 hyonun321 added FE 프론트엔드 작업 BE 백엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 P2 일반 우선순위 labels Nov 30, 2024
@hyonun321 hyonun321 self-assigned this Nov 30, 2024
Copy link
Collaborator

@minjungw00 minjungw00 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
Collaborator

@Ludovico7 Ludovico7 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

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!!!!!!!!!!! 👍

@github-actions github-actions bot merged commit c2eca98 into dev Dec 2, 2024
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 작업 FE 프론트엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 P2 일반 우선순위
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants