-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Feature] 크리스마스 보드, 폴라로이드 테마 추가, Google Tag Manager 설정 #139
Conversation
src/lib/constants/polaroidConfig.ts
Outdated
@@ -13,6 +13,50 @@ export const THEMAS: Record<ThemaKeyType, ThemaType> = { | |||
descriptionStyle: | |||
'linear-gradient(180deg, rgba(255, 255, 255, 0.20) 10.71%, rgba(255, 255, 255, 0.50) 57.96%, rgba(255, 255, 255, 0.00) 100%), #eaeaea', | |||
}, | |||
'F-12': { |
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.
여기 순서는 lint가 자동으로 한건가요? 아니라면 0부터 순서대로되면 좋을것 같습니다!
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.
디자인팀에서 정해준 폴라로이드 순서가 변경돼서 이렇게 바꿨는데, 순서대로 하려면 파일 이름을 모두 변경해야 합니다. 더 좋은 방법이 있을까요??
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.
아 순서때문에 그런거군요. 순서가 중요하다면 Object.entries 대신 보여줄 테마 키값을 별도의 배열로 관리하는게 나을 것 같은데 어떨까요?
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 👍
👀 관련 이슈
크리스마스
✨ 작업한 내용
🌀 PR Point
고칠 부분이 있으면 알려주세요!
🍰 참고사항
NEXT_PUBLIC_GTM_ID 추가해주세요 디스코드에 있습니다!
📷 스크린샷 또는 GIF