-
Notifications
You must be signed in to change notification settings - Fork 1
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
[TMN-58] 회원가입 후 온보딩 #642
[TMN-58] 회원가입 후 온보딩 #642
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
수고하셨습니다!
}, | ||
]; | ||
|
||
export const RECOMMENDATION_REAL = [ |
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.
process.env.node_env 애 떠러 REAL을 보여줄지 말지 컨트롤 하는건 어떨까요
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.
node_env 인 dev 배포환경일때는 'production'으로 보이지 않나요?
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.
NEXT_PUBLIC_ENV 라는 환경변수를 따로 만들게요
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.
}; | ||
|
||
const handleComplete = async () => { | ||
await Promise.all( |
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.
👍
src/app/onboarding/page.tsx
Outdated
key={props.id.toString()} | ||
id={props.id} | ||
tags={props.tags} | ||
nickname={props.nickname} | ||
profileImageUrl={props.profileImageUrl} |
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.
{...props}를 이용하면 어떨까여
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.
rightComponent?: React.ReactNode; | ||
} | ||
|
||
function CenterTextHeader({ title, rightComponent }: CenterTextHeaderProps) { |
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.
일단 임시로 사용하는 Header인거죠?
나중에도 사용하게 된다면 디자인 시스템에 추가하면 좋을 것 같네요
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.
네 빠르게 작업하기 위해서 추가한 컴포넌트입니다
width: '33%', | ||
}); | ||
|
||
const centerCss = css({ |
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.
css 대신 center pattern 이용해도 좋을 것 같아요~
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.
🤔 해결하려는 문제가 무엇인가요?
https://tenminute.atlassian.net/browse/TMN-36?atlOrigin=eyJpIjoiODg4MWI5Zjk2NTQ3NDA0MjkwNzcxMjU0MGFhOTljMWYiLCJwIjoiaiJ9
🎉 변경 사항
🙏 여기는 꼭 봐주세요!
사용 방법
🌄 스크린샷
2024-03-27.12.10.48.mov
📚 참고