Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@NYeonK
Copy link
Contributor

@NYeonK NYeonK commented Aug 25, 2023

  • 브랜치명, 브랜치 알맞게 설정
  • Reviewer, Assignees, Label, Milestone, Issue(PR 작성 후에) 붙이기
  • PR이 승인된 경우 해당 브랜치는 삭제하기

📌 내용

  • 이벤트용 배너를 구현했습니다.
  • 텍스트를 제외하고는 배경이미지로 넣었는데요! 화질 QA가 생기면 디자인에 다시 작업 요청할 예정입니다.
  • 이 PR 먼저 머지하고, 이벤트 branch에 진입 경로 추가하면 될 것 같아요!

📸 스크린샷

@NYeonK NYeonK requested a review from ilmerry August 25, 2023 20:27
@NYeonK NYeonK self-assigned this Aug 25, 2023
@NYeonK NYeonK linked an issue Aug 25, 2023 that may be closed by this pull request
4 tasks
Copy link
Member

@ilmerry ilmerry left a 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}>
Copy link
Member

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}

로 해주시면 될 것 같습니다!

Copy link
Contributor Author

@NYeonK NYeonK Aug 26, 2023

Choose a reason for hiding this comment

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

넵 반영해서 먼저 머지할게요~! 경로 진입 잘되는지 이따 확인부탁드릴게요:)

Comment on lines +14 to +19
<St.Content className={GTM_CLASS_NAME.mainBannerEvent} islogin={isLogin}>
이벤트 참여하고 치킨 받아가세요!
</St.Content>
<St.SubContent className={GTM_CLASS_NAME.mainBannerEvent} islogin={isLogin}>
피클의 미래 모습을 체험해 볼 수 있어요!
</St.SubContent>
Copy link
Member

Choose a reason for hiding this comment

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

여기서의 islogin은 반드시 true로 들어가지 않나요?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵ㅎㅎ 기존 띠배너랑 동일한 태그를 썼기 때문에, 폰트 색 구분을 위헤 넣은 값입니다!

@NYeonK NYeonK merged commit 4921737 into release/2.1.0 Aug 26, 2023
@NYeonK NYeonK deleted the feat/#330-event_banner branch August 26, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Event ] 이벤트용 띠배너 구현

3 participants