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

refactor: zustand 설치 및 일부 적용 #299

Closed
wants to merge 4 commits into from

Conversation

glowisn
Copy link
Collaborator

@glowisn glowisn commented Apr 8, 2024

Checklist

  • Code Review: 작성한 코드를 다시 한 번 꼼꼼이 확인했나요?
  • Testing: 앱이 잘 구동되는지 개발한 기능이 문제 없이 작동하는지 확인했나요?
  • Remove: print나 주석 등 필요없는 코드를 삭제했나요?
  • Conflict Resolution: 충돌을 해결하는 과정을 거쳤나요?
  • New Dependencies: 새로운 dependency를 추가했나요?

Description

zustand 추가 후 RoomContext에서 분리 가능한 부분만 일단 분리했습니다.

useRef를 쓰는 경우는 zustand store에 넣어서 useRef로 부르면 되는 것 같은데, 너무 복잡해져서 일단 contextAPI 채로 나뒀습니다.
참고링크

isHost, roomCode 는 서버 상태로 분리 할 예정이라 일단 원래 ussRoom 사용하는 RoomContext 상태로 나뒀습니다.

그리고 RoomContext에서 바로 내려지는 상태들도 있는데, 그 상태를 받아 다시 하위 컴포넌트에 props로 내리는 형태의 코딩이 많이 되어있습니다. 저희가 Context API를 사용하면서 필요없는 선언부를 줄이기 위해 사용한 방식인데, zustand를 쓰면 그것도 해결할 수 있지 않나 싶습니다.

Changes Made

  • zustand dependecy 추가
  • RoomContex 및 사용처 다수

Extra Comments

Demo

@glowisn glowisn self-assigned this Apr 8, 2024
@glowisn glowisn linked an issue Apr 8, 2024 that may be closed by this pull request
@glowisn
Copy link
Collaborator Author

glowisn commented Apr 8, 2024

지금 버그가 있는데 추후 수정하겠습니다.

수정했습니다.

@glowisn glowisn marked this pull request as ready for review April 29, 2024 06:09
@glowisn glowisn closed this May 3, 2024
@glowisn
Copy link
Collaborator Author

glowisn commented May 3, 2024

브랜치에 한글 이름이 있어 새로 PR 파겠습니다

@glowisn glowisn deleted the 297-refactor-roomcontext-zustand-이사 branch May 3, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor : RoomContext -> Zustand 이사
3 participants