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.

Sign up for GitHub

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

Jump to bottom

[Add] #9 - 에셋 추가 및 리드미 수정 #11

Merged
merged 4 commits into from
Jul 4, 2024
Merged

[Add] #9 - 에셋 추가 및 리드미 수정 #11

merged 4 commits into from
Jul 4, 2024

Conversation

mcrkgus
Copy link
Member

@mcrkgus mcrkgus commented Jul 2, 2024 •
edited
Loading

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • Asset 추가
  • 리드미 수정

✅ Check List

  • Merge 하는 브랜치가 올바른가?
  • 코딩 컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?

📟 관련 이슈

@mcrkgus mcrkgus added + Add 라이브러리 추가, 에셋 추가 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일 💞 가현 가현 공주 작업 labels Jul 2, 2024
@mcrkgus mcrkgus self-assigned this Jul 2, 2024
shimseohyun
shimseohyun approved these changes Jul 2, 2024
View reviewed changes
Copy link
Contributor

@shimseohyun shimseohyun left a comment

Choose a reason for hiding this comment

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

에셋 추가 감사합니다 너무너무 고생했어요!

README.md
@@ -139,4 +139,4 @@ P3 : 단순 의견 제시 (무시해도 됩니다)
| SnapKit | `SPM` |Auto Layout 제약조건을 코드로 쉽게 작성할 수 있도록 한다.|
| Kingfisher | `SPM` |URL로부터 이미지 다운 중 처리 작업을 간소화할 수 있도록 한다.|
| MapKit | `SPM` |다양한 지도 기능을 원활하게 구현할 수 있도록 한다.|
| Moya | `SPM` | 네트워크 요청을 간현화하고, 구조화된 방식으로 관리하여 코드의 가독성과 유지 보수성을 높인다.|
| Moya | `SPM` | 네트워크 요청을 간편화하고, 구조화된 방식으로 관리하여 코드의 가독성과 유지 보수성을 높인다.|
Copy link
Contributor

Choose a reason for hiding this comment

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

앗 나의 실수 🥹🥹🥹

EunsuSeo01
EunsuSeo01 approved these changes Jul 3, 2024
View reviewed changes
Copy link
Member

@EunsuSeo01 EunsuSeo01 left a comment

Choose a reason for hiding this comment

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

고생했어용

...ijogbo/Hankkijogbo/Global/Resources/Assets/Colors.xcassets/hankkiMain.colorset/Contents.json Outdated
Copy link
Member

Choose a reason for hiding this comment

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

스크린샷 2024-07-03 오전 11 49 02 스크린샷 2024-07-03 오전 11 53 10

P2: 디자인 분들이 작업하신 뷰에서 노란색 Sub Color들을 확인하면 색상 이름이 이런 식으로 적혀 있어서 이 이름을 따라가면 작업할 때 좋을 것 같아요~!

Copy link
Member Author

Choose a reason for hiding this comment

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

기존의 yellow를 main으로, lightyellow를 sub로 하자는건가유?

@mcrkgus mcrkgus merged commit 7703de9 into develop Jul 4, 2024
@mcrkgus mcrkgus deleted the add/#9 branch July 4, 2024 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shimseohyun shimseohyun shimseohyun approved these changes

@EunsuSeo01 EunsuSeo01 EunsuSeo01 approved these changes

Assignees

@mcrkgus mcrkgus

Labels
+ Add 라이브러리 추가, 에셋 추가 💞 가현 가현 공주 작업 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Add] Color, Icon Asset Setting
3 participants
@mcrkgus @EunsuSeo01 @shimseohyun