Skip to content
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

feat: SemanticBGColor 추가 #7

Merged
merged 1 commit into from
Sep 27, 2023
Merged

feat: SemanticBGColor 추가 #7

merged 1 commit into from
Sep 27, 2023

Conversation

HyunsDev
Copy link
Contributor

1️⃣ 어떤 작업을 했나요? (Summary)

#6 (comment)

기존 코드에 영향을 미치지 않는 변경사항

SemanticBGColor 타입을 추가했어요.

2️⃣ 알아두시면 좋아요!

SemanticBGColor 타입은 기존 타입에서 자주 사용할 것 같은 타입을 분리하기 위한 유틸리티 타입이에요. 별도의 색상 파레트를 가지는 것이 아닌 *BG 형태의 색상만 별도로 분리했어요.

3️⃣ 추후 작업

Copy link
Collaborator

@nijuy nijuy left a comment

Choose a reason for hiding this comment

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

👍 Badge에서 유용하게 쓰겠군요...... 고생하셨습니닥!!!!

@nijuy nijuy mentioned this pull request Sep 26, 2023
@nijuy
Copy link
Collaborator

nijuy commented Sep 27, 2023

@HyunsDev 여기 PR!! 머지 부탁드립니다 👀 되는대로 Badge에 적용해볼게요 (╯▽╰ )

@Hanna922
Copy link
Member

@nijuy 저도 확인했어요! 저희 요런(어프로브 받은) PR은 아무나 머지해도 될 것 같은데 바로 머지하시는거 어떠신가용? 작업이랑 연관된 거라서욤

@nijuy
Copy link
Collaborator

nijuy commented Sep 27, 2023

@nijuy 저도 확인했어요! 저희 요런(어프로브 받은) PR은 아무나 머지해도 될 것 같은데 바로 머지하시는거 어떠신가용? 작업이랑 연관된 거라서욤

이번 pr은 저저번 회의에서 나왔던 내용이고, 작업한 라인 수도 적어서 둘다 확인했다면 현스말고 다른 사람이 머지해도 될 거 같아요!! ^ㅅ^ (제가 이 코멘트 달고 할게용)

저는 말씀해주신 방식(어푸룹한 사람이 추가 수정사항 없다면 머지) 좋아요! 현스 의견은 어떤지 코멘트나 담 회의때 같이 말해봐요 🤤

@nijuy nijuy merged commit 309b129 into develop Sep 27, 2023
@Hanna922 Hanna922 deleted the feat/color branch September 27, 2023 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants