-
Notifications
You must be signed in to change notification settings - Fork 0
feat(frontend) : 캐스팅 완료 페이지 레이아웃 구현 #19
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
Figma 시안 기반 캐스팅 완료 화면을 풀스크린 레이아웃으로 구현하고, 9개 언어(en, es, id, ja, ko, th, tl, vi, zh) castingComplete 키 추가
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은 연습생 캐스팅 완료 화면을 구현하고 관련 네비게이션을 추가하는 작업을 잘 수행했습니다. 새로운 CastingCompletePage 페이지와 CastingComplete 프레젠테이셔널 컴포넌트가 아키텍처 원칙에 따라 잘 분리되어 구현되었습니다. 또한, 스토리북 테스트 케이스와 9개 언어에 대한 다국어 지원이 포함된 점도 좋습니다. 코드 전반적으로 프로젝트의 스타일 가이드와 네이밍 컨벤션을 잘 따르고 있습니다. 한 가지 개선점으로, 현재 페이지에서 사용 중인 목업 데이터를 실제 동적 데이터로 교체할 필요가 있다는 점을 리뷰 코멘트로 남겼습니다.
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.
LGTM
song-eun
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.
확인했습니다! 감사합니다
요약
연습생 캐스팅 후 표시되는 캐스팅 완료 화면을 구현하고, 캐스팅 목록/캐스팅 보드로의 네비게이션을 연결하였습니다.
목적
캐스팅 플로우에서 연습생 캐스팅 직후 결과를 확인하고, 다음 행동(연습생 탐색 또는 캐스팅 보드 확인)으로 이어질 수 있도록 합니다.
주요 변경 사항
캐스팅 완료 페이지
컴포넌트
다국어
스토리북
테스트 체크리스트