-
Notifications
You must be signed in to change notification settings - Fork 0
feat(frontend): 그룹 기획 페이지 구현 #21
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
그룹 기획 페이지와 관련 컴포넌트, 다국어 리소스, Storybook 스토리를 추가하는 Pull Request 잘 보았습니다. 전반적으로 기능 구현이 잘 이루어졌습니다. 몇 가지 개선점을 제안합니다. 특히, 스타일 가이드 및 관련 규칙에 따라 일부 컴포넌트에서 하드코딩된 색상 값을 DaisyUI 시맨틱 토큰으로 변경하는 것이 필요합니다. 또한, Storybook에서 alert 대신 addon-actions를 사용하고, 다국어 파일의 플레이스홀더에 더 구체적인 예시를 추가하여 개발자 경험과 사용자 경험을 개선할 수 있습니다. 코드 가독성을 높일 수 있는 부분에 대한 제안도 포함했습니다. 자세한 내용은 각 파일의 주석을 참고해 주세요.
frontend/src/stories/group-creation/GroupPlanningForm.stories.tsx
Outdated
Show resolved
Hide resolved
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!
zhwltlr
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.
확인했습니다!
요약
포지션 배정 완료 후 그룹 이름과 엠블럼을 설정하는 2단계 그룹 기획 페이지를 구현했습니다.
목적
사용자가 그룹 이름을 짓고 AI로 엠블럼 이미지를 생성할 수 있습니다.
변경 사항
그룹 기획 페이지
/complete페이지로 이동컴포넌트 구조
GroupPlanningForm: 메인 폼 컴포넌트 (스텝 관리, 버튼)EmblemStep: Step 2 엠블럼 생성 컴포넌트 분리스타일 수정
Header:shrink-0추가 (화면 높이 작을 때 헤더 높이 유지)StepCard:w-full추가테스트