대기실 페이지 2차 작업 #114
Open
대기실 페이지 2차 작업 #114
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- add, reset 함수도 추가 - 선택을 완료한 플레이어들의 이름을 저장한다.
- 게임 시작 시 UserInfoCard 색상 원래대로 변경
…pHi/grouphi-mvp-fe into GRPHI-133-feat/WaitingRoomSocket2
- handleEnterNextRound로 함수 통합 - useEffect문에 if문 추가
- api 폴더 생성 및 기존 roomDetail 훅 이동
- 중복 로직 쿼리로 통합 - refetch 이용해 쿼리 데이터 업데이트 - 다음 라운드 이동 시 쿼리 데이터 삭제
- 겹치는 로직 하나로 통일 - 쿼리 불필요해져서 다시 삭제
- page가 너무 커져서 분리
- 추후에 새로운 게임 추가되면 그에 맞춰 변경 될 예정 - UserList section 디자인 이관
- 변경 후 방 상세조회 시 바뀐 이름으로 유저가 새로 생기는 문제 발생하므로 해결 필요
…into GRPHI-133-feat/WaitingRoomSocket2
6 tasks
…into GRPHI-133-feat/WaitingRoomSocket2
joyswim
reviewed
Feb 13, 2025
Comment on lines
+54
to
+57
| <div className="fixed w-full h-screen -z-10"> | ||
| <StarsBackground /> | ||
| <ShootingStars /> | ||
| </div> |
There was a problem hiding this comment.
[기록용] 별똥별 배경이 다른 컴포넌트 위로 올라오는 이슈 수정
joyswim
reviewed
Feb 13, 2025
| /> | ||
| {votes1 !== 0 && ( | ||
| <BarItem | ||
| className={clsx('bg-primary', votes2 === 0 && 'rounded-r-full')} |
There was a problem hiding this comment.
clsx 사용보다 cn 유틸함수를 사용해보는 건 어떨까요? cn 유틸함수는 clsx, twMerge를 조합한 함수입니다.
clsx는 조건부 className을 하나로 합쳐주는 역할입니다.twMerge는 중복되거나 불필요한 TailwindCSS className을 제거하여 최적화합니다.
joyswim
reviewed
Feb 13, 2025
Comment on lines
+170
to
+175
| setChatMessages(() => [ | ||
| { | ||
| sender: SOCKET.SYSTEM, | ||
| content: '게임이 종료되었습니다.', | ||
| }, | ||
| ]); |
joyswim
reviewed
Feb 13, 2025
Comment on lines
+181
to
+187
| case SOCKET.TYPE.ERROR: | ||
| toast({ | ||
| variant: 'destructive', | ||
| title: '문제가 생겼습니다. 다시 시도해주세요.', | ||
| }); | ||
| router.push(PATH.HOME); | ||
| break; |
joyswim
reviewed
Feb 13, 2025
ksjdev
requested changes
Feb 15, 2025
Comment on lines
+18
to
+28
| const handleEnterNextRound = async () => { | ||
| sendMessage({ | ||
| destination: `${SOCKET.ENDPOINT.BALANCE_GAME.NEXT}`, | ||
| }); | ||
| }; | ||
|
|
||
| const handleMoveToWaitingRoom = () => { | ||
| sendMessage({ | ||
| destination: `${SOCKET.ENDPOINT.BALANCE_GAME.END}`, | ||
| }); | ||
| }; |
There was a problem hiding this comment.
P3;
async 키워드가 사용되지 않아도 되는 곳에 붙어있는 것 같습니다.
만약 client.current?.publish 로직을 사용하는데에 async 키워드가 필요하다면 통일해주시면 좋을 것 같아요~
| /> | ||
| {votes2 !== 0 && ( | ||
| <BarItem | ||
| className={clsx('bg-secondary', votes1 === 0 && 'rounded-l-full')} |
| )} | ||
| {players.map((data, index) => ( | ||
| <UserInfoCard | ||
| key={index} |
There was a problem hiding this comment.
P1;
생각해보니 키값으로 인덱스를 사용해도 괜찮을까요? 유저 목록의 변동이 잦은 경우, 잘못된 값을 재사용할 가능성이 존재할 것 같다고 느껴지는데 어떻게 생각하시나요?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝작업 내용
이슈 내역
📷스크린샷(필요 시)
2025-02-04.9.29.12.mov
전원 선택 시 바로 결과 창으로 이동한다.
2025-02-04.9.34.12.mov
게임 종료 시 채팅창을 초기화 한다.
✨PR Point