Skip to content

Conversation

@zhwltlr
Copy link
Collaborator

@zhwltlr zhwltlr commented Jan 30, 2026

요약

연습생 캐스팅 후 표시되는 캐스팅 완료 화면을 구현하고, 캐스팅 목록/캐스팅 보드로의 네비게이션을 연결하였습니다.

목적

캐스팅 플로우에서 연습생 캐스팅 직후 결과를 확인하고, 다음 행동(연습생 탐색 또는 캐스팅 보드 확인)으로 이어질 수 있도록 합니다.

주요 변경 사항

캐스팅 완료 페이지

  • 캐스팅 완료 풀스크린 페이지 (/[aidolId]/casting-complete) 신규 구현
  • 하트 아이콘, 완료 메시지, 남은 캐스팅 슬롯 수 표시
  • '다음 연습생 찾기' → /aidols/[aidolId]/casting, '캐스팅 보드 보기' → /[aidolId]/casting-board 네비게이션

컴포넌트

  • CastingComplete Presentational 컴포넌트 신규 추가

다국어

  • castingComplete 번역 키 추가 (9개 언어)

스토리북

  • CastingComplete.stories.tsx 추가 (Default/FiveSlots/ZeroSlots)

테스트 체크리스트

  • 캐스팅 완료 화면이 풀스크린으로 중앙 정렬되어 표시되는지 확인
  • 남은 캐스팅 슬롯 수가 정상적으로 표시되는지 확인
  • '다음 연습생 찾기' 버튼 클릭 시 캐스팅 목록 페이지로 이동 확인
  • '캐스팅 보드 보기' 버튼 클릭 시 캐스팅 보드 페이지로 이동 확인

@zhwltlr zhwltlr changed the title Feat/casting complete 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은 연습생 캐스팅 완료 화면을 구현하고 관련 네비게이션을 추가하는 작업을 잘 수행했습니다. 새로운 CastingCompletePage 페이지와 CastingComplete 프레젠테이셔널 컴포넌트가 아키텍처 원칙에 따라 잘 분리되어 구현되었습니다. 또한, 스토리북 테스트 케이스와 9개 언어에 대한 다국어 지원이 포함된 점도 좋습니다. 코드 전반적으로 프로젝트의 스타일 가이드와 네이밍 컨벤션을 잘 따르고 있습니다. 한 가지 개선점으로, 현재 페이지에서 사용 중인 목업 데이터를 실제 동적 데이터로 교체할 필요가 있다는 점을 리뷰 코멘트로 남겼습니다.

@zhwltlr zhwltlr self-assigned this Jan 30, 2026
@zhwltlr zhwltlr marked this pull request as ready for review January 30, 2026 15:30
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.

LGTM

Copy link
Collaborator

@song-eun song-eun left a comment

Choose a reason for hiding this comment

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

확인했습니다! 감사합니다

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