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/#181 socketStore에 accessToken의존성 부여 #182

Conversation

hyonun321
Copy link
Collaborator

@hyonun321 hyonun321 commented Nov 24, 2024

📝 변경 사항

2f2d12d2.mp4
  • 로그인/로그아웃 할때 워크스페이스가 갱신됨.

  • socket 생성때 auth에서 사용한 accessToken을 활용

  • null 일경우 guestWorkspace를 표현할 예정

🔍 변경 사항 설명

  • socketStore가 실행될때, JWT 에서 사용되는 accessToken을 가져오고 여기에 의존성을 부여합니다.
  • 기존에
if(socket?.connected)  return;

를 삭제했습니다. -> 소켓이 연결돼있으면 더이상 안하는 코드여서 삭제했습니다.

🙏 질문 사항

  • 추후 workspace, guest 별로 workspace를 나눌때 commit 사항으로 포함해도 되지만, 먼저 dev에 붙이고 추후 socket 과 workspace연동 작업을 진행하는게 낫다고 판단하여 간단하게 PR을 올렸습니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

- socket 생성때 auth에서 사용한 accessToken을 활용
- null 일경우 guestWorkspace를 표현할 예정

#181
@hyonun321 hyonun321 added the FE 프론트엔드 작업 label Nov 24, 2024
@hyonun321 hyonun321 self-assigned this Nov 24, 2024
@hyonun321
Copy link
Collaborator Author

user의 접속 ID별로 해보도록 수정

@hyonun321
Copy link
Collaborator Author

해당 PR은 이미 다른 브런치에 작업한 사항이 있어 close함

@hyonun321 hyonun321 closed this Nov 25, 2024
@hyonun321 hyonun321 deleted the Feature/#181_accessToken이_변경되면_socket을_다시_설정하도록_수정 branch November 25, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant