Skip to content

Conversation

@song-eun
Copy link
Collaborator

@song-eun song-eun commented Jan 30, 2026

요약

캐스팅된 멤버들에게 포지션을 배정하는 페이지를 구현했습니다. 카드를 탭하면 모달에서 드롭다운으로 포지션을 선택할 수 있습니다.

목적

그룹 멤버 구성 후 각 멤버에게 역할(메인 보컬, 메인 댄서 등)을 배정하는 기능이 필요합니다. 모든 멤버의 포지션 배정이 완료되면 다음 단계(그룹 기획)로 진행할 수 있습니다.

변경 사항

포지션 배정 페이지 (/aidols/{aidolId}/position-assignment)

  • DaisyUI 드롭다운 컴포넌트 사용
  • 이미 배정된 포지션은 "(멤버이름)" 표시 및 선택 비활성화
  • 모든 멤버 배정 완료 시 "그룹 기획하기" 버튼 활성화

컴포넌트

  • PositionBoard: 메인 화면 (카드 그리드 + 하단 버튼)
  • PositionSelector: 포지션 드롭다운 선택
  • Card: 조건문 단순화 및 position variant 지원
  • ProfileContent: showBiography prop 추가

스키마

  • POSITIONS 상수 및 Position 타입 추가
  • CompanionUpdateposition 필드 추가

기타

  • Lottie 애니메이션 경로 상수 분리
  • Storybook 스토리 추가

테스트

  • 드롭다운에서 포지션 선택 가능 확인
  • 이미 배정된 포지션이 "(멤버이름)"으로 표시되고 선택 불가 확인
  • 모든 멤버 배정 완료 시 버튼 활성화 확인
  • Storybook에서 WithModalOpen 스토리 확인

@song-eun song-eun changed the title Feat/position page feat(frontend): 포지션 배정 페이지 구현 Jan 30, 2026
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 PR은 멤버에게 포지션을 배정하는 새로운 페이지와 관련 컴포넌트들을 추가합니다. 전반적으로 기능 구현이 잘 이루어졌으며, Storybook 테스트 케이스와 다국어 지원이 포함된 점이 좋습니다. 코드 재사용성을 높이기 위해 constants/assets.ts 파일을 추가하고 Card 컴포넌트를 리팩토링한 부분도 긍정적입니다. 다만, 코드 품질과 유지보수성을 더욱 향상시키기 위해 몇 가지 개선점을 제안합니다. 주요 내용은 다음과 같습니다:

  • 스타일 가이드에 따른 시맨틱 색상 토큰 사용
  • 컴포넌트 간의 역할 분리 개선
  • 타입 안정성 강화
  • 국제화(i18n) 처리 방식 개선

@song-eun song-eun marked this pull request as ready for review January 30, 2026 16:49
@song-eun song-eun requested a review from zhwltlr January 30, 2026 22:48
Copy link
Contributor

@ywkim ywkim left a comment

Choose a reason for hiding this comment

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

🚀

@song-eun song-eun self-assigned this Jan 31, 2026
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