[#78]: 내 정보 불러오기 api 연동

[zzzRYT]

📝 변경 사항

🔗 연관된 이슈

Closes #78

📱 스크린샷 (UI 변경 시)

Before

After

• 기존의 room은 줄여도 3단계 반응형으로 명확하게 나타나지 않음
• 해당 부분을 3단계 반응형 형식으로 줄일 수 있도록 기존의 useResponsive 훅을 사용해서 재구축

📋 추가 정보

• Lobby에 적용되어있던 반응형 layout을 room에도 적용했습니다.

리뷰 포인트

배포 시 주의사항

Breaking Changes

[github-actions]

🤖 자동 감지된 연관 이슈들이 PR 템플릿에 자동으로 추가되었습니다

감지된 이슈: #78

브랜치명: feature-78/get-accounts-me-api

브랜치명과 커밋 메시지를 분석하여 자동으로 연결했습니다. 수정이 필요하시면 PR 설명을 직접 편집해주세요.

[soonki-98]

[제안] 두 레이아웃 컴포넌트의 코드가 크게 다르지 않고 중복이 발생하니 useResponsive 훅을 사용해서 반응형 컴포넌트를 분기 태우는것 보다는 useResponsiveClasses 훅을 사용해서 하나의 컴포넌트에서 반응형 클래스만 넘기는것은 어떨까요?

나중에 ui가 어떻게 바뀔지 모르지만 지금은 동일한 태그 구조 아래에서 classname만 다르니 그렇게 구성해도 충분할거라고 생각이 들었습니다.

[zzzRYT]

나중에 ui가 어떻게 바뀔지 모르지만 지금은 동일한 태그 구조 아래에서 classname만 다르니 그렇게 구성해도 충분할거라고 생각이 들었습니다.

Lobby > index.tsx와 일관된 형식으로 작성하려고 이렇게 한건데. 사실 작업하면서도 너무 동일한 컴포넌트인데 나눠야 하나? 이런생각을 하긴 했습니다. 그럼에도 나눈 이유는

• 현재 room > index.tsx에서 많은 데이터를 처리하는 느낌이 이었습니다. 나중에 handleStartGame같은 함수도 완성되면 더욱 복잡도가 올라갈 것으로 생각되었습니다.
• 이 글을 적으면서 느낀건데... 약간 괜한 걱정인 것 같네요😅 나중에 hook이나 함수로 쪼개는게 더 현명해 보입니다. 제안주신 내용으로 수정해서 반영하겠습니다!

[zzzRYT]

@soonki-98 이전 작업에서 진행한 loabby > accounts로직을 lobby에서 불러오는게 아니라, accounts에서 불러오게 하고싶은데, 해당 PR에서 작업을 해도 괜찮을까요?

[soonki-98]

이전 작업에서 진행한 loabby > accounts로직을 lobby에서 불러오는게 아니라, accounts에서 불러오게 하고싶은데, 해당 PR에서 작업을 해도 괜찮을까요?

새로 이슈를 만들고 새로운 PR에서 작업해주셔요