-
Notifications
You must be signed in to change notification settings - Fork 3
Feat/#330 event banner #331
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
Conversation
ilmerry
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.
빠른 작업 👍👍 고생많으셨어요!!
|
|
||
| if (isLogin) { | ||
| return ( | ||
| <St.EventContainer to="" className={GTM_CLASS_NAME.mainBannerEvent}> |
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.
여기서 바로 머지해도될 것 같아요! /card-collection?type=event로 이동해야하니까
${routePaths.CardCollection}?type=${LocationType.EVENT}로 해주시면 될 것 같습니다!
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.
넵 반영해서 먼저 머지할게요~! 경로 진입 잘되는지 이따 확인부탁드릴게요:)
| <St.Content className={GTM_CLASS_NAME.mainBannerEvent} islogin={isLogin}> | ||
| 이벤트 참여하고 치킨 받아가세요! | ||
| </St.Content> | ||
| <St.SubContent className={GTM_CLASS_NAME.mainBannerEvent} islogin={isLogin}> | ||
| 피클의 미래 모습을 체험해 볼 수 있어요! | ||
| </St.SubContent> |
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.
여기서의 islogin은 반드시 true로 들어가지 않나요?!
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.
넵ㅎㅎ 기존 띠배너랑 동일한 태그를 썼기 때문에, 폰트 색 구분을 위헤 넣은 값입니다!
📌 내용
📸 스크린샷