-
Notifications
You must be signed in to change notification settings - Fork 0
feat(frontend): 포지션 배정 페이지 구현 #20
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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) 처리 방식 개선
ywkim
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
요약
캐스팅된 멤버들에게 포지션을 배정하는 페이지를 구현했습니다. 카드를 탭하면 모달에서 드롭다운으로 포지션을 선택할 수 있습니다.
목적
그룹 멤버 구성 후 각 멤버에게 역할(메인 보컬, 메인 댄서 등)을 배정하는 기능이 필요합니다. 모든 멤버의 포지션 배정이 완료되면 다음 단계(그룹 기획)로 진행할 수 있습니다.
변경 사항
포지션 배정 페이지 (/aidols/{aidolId}/position-assignment)
컴포넌트
PositionBoard: 메인 화면 (카드 그리드 + 하단 버튼)PositionSelector: 포지션 드롭다운 선택Card: 조건문 단순화 및positionvariant 지원ProfileContent:showBiographyprop 추가스키마
POSITIONS상수 및Position타입 추가CompanionUpdate에position필드 추가기타
테스트
WithModalOpen스토리 확인