Skip to content

Conversation

@lnxhigh
Copy link
Collaborator

@lnxhigh lnxhigh commented Jan 7, 2026

📋 작업 범위 (Scope)

  • Frontend (React)
  • Backend (NestJS)
  • Common (Shared Types, Utils)
  • Infrastructure (DevOps, CI/CD, Docker)
  • Documentation (README, Wiki)

🔗 관련 이슈 (Related Issue)

🛠️ 작업 내용 (Description)

  • 메인 페이지 UI

    • Hero + Action Cards Section + Feature Cards Section 으로 구성됨
      • Action Cards 는 방 만들기 카드, 방 번호로 입장 카드를 의미함
      • Feature Cards 는 서비스의 기능을 표현하는 카드를 의미함
    • 특징
      • 중앙 정렬을 적극적으로 사용함
      • 바탕 화면을 흰색으로 두고 가벼운 색을 사용함
  • Room REST API

    • 클라이언트에 Room 관련 API 를 작성함
    • 서버에 Room 으로 Redirect 하는 API 를 작성함
      • Get ':roomCode/join'
    • ROOM API Prefix 를 'api/room' 에서 'api/rooms' 로 변경함
    • Room code 가 소문자를 사용하지 않도록 수정함
      • 코드를 직접 입력하는 경우 대소문자를 구분하게 되면 UX 가 불편함

📦 패키지 변경 사항 (Dependencies)

  • 없음
  • ## 있음 (아래에 기입)

📸 스크린샷

image

실패 시 다음과 같이 에러를 표시함

image

✅ 체크리스트 (Self Checklist)

  • 코드가 정상적으로 빌드/실행되는지 확인했나요?

💬 Remarks

// local storage key
key = `ptId:${roomCode}`;

@lnxhigh lnxhigh marked this pull request as ready for review January 8, 2026 15:31
Comment on lines 10 to 12
<Route path="/" element={<HomePage />} />
<Route path="/room/prototype" element={<RoomPage />} />
<Route path="*" element={<NotFoundPage />} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 path가 이제 /room/:roomCode 이런식으로 되어야 하지 않나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

그러네요 이 부분 수정하겠습니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

클라이언트에서 room id 를 사용하고 있는 부분이 많은데 store 부분은 머지한 코드에서 다시 수정하는 게 나을 것 같습니다
Room path 는 /rooms/:roomCode 로 수정했습니다 (관례적으로 rooms 를 많이 사용해서 복수형을 사용했습니다)

@lnxhigh lnxhigh merged commit 01f4ea0 into dev Jan 8, 2026
@inaemin inaemin deleted the feat/#68 branch January 8, 2026 17:15
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.

4 participants