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

fix: 기본 명함 앞면 수정 (#353) #354

Merged
merged 6 commits into from
Jan 31, 2023
Merged

fix: 기본 명함 앞면 수정 (#353) #354

merged 6 commits into from
Jan 31, 2023

Conversation

hyun99999
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 이미지 에셋 추가(생일, mbti, 전화번호)
  • 전화번호 추가
  • 생일, mbti 수정

서버 API 가 나오고도 회의를 거쳐봐야할거 같고 그 이후에도 다시금 만질 코드가 있을 것 같습니당.
꽤나 복잡하게 구현되어 있고, 셀안에 셀에서 델리게이트로 전달하는 등의 구조가 있습니당! 예전 코드를 다시 수정하고 중간중간 리펙토링하면서 진행하니 진땀이 나기도 하네여.. 화이팅 해봅시더..

🚨 참고사항

  • 홈에서는 서버에서 전화번호를 보내주지 않습니다. Card 데이터모델에서 전화번호가 String? 자료형이기때문에 제가 작성한 로직때문에 전화번호 이미지 에셋이 보입니당. 이는 코드에 주석처리한대로 봐주시면 될 것 같습니다.

  • 팀 나다에서도 적용되고 있습니다.(수동으로 Card 객체를 전달하기 때문에 이때는 전화번호 이미지 에셋이 안보임)

📸 스크린샷

기능 스크린샷
미리보기(3개)
미리보기(2개)
미리보기(1개)

📮 관련 이슈

@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Jan 30, 2023
@hyun99999 hyun99999 self-assigned this Jan 30, 2023
@hyun99999 hyun99999 changed the title fix: 기본 명함 앞면 수정 fix: 기본 명함 앞면 수정 (#353) Jan 30, 2023
dlwns33
dlwns33 approved these changes Jan 31, 2023
View reviewed changes
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

수고하셨씁니다!

NADA-iOS-forRelease/Sources/Cells/CreationCard/FrontCardCreationCollectionViewCell.swift Outdated
@@ -114,7 +115,7 @@ extension FrontCardCreationCollectionViewCell {
birthView.backgroundColor = .textBox
birthLabel.font = .textRegular04
birthLabel.textColor = .quaternary
birthLabel.text = "생년월일"
birthLabel.text = "생일"
Copy link
Member

Choose a reason for hiding this comment

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

요거는 그냥 생각나서 이곳에 적어봅니당 !
하지 않을수도 있지만 나중에 localization을 하게 되면 요런것들이 다 영어로 보여야 하잖아요..?
그래서 요런 것들을 StringLiterals에 모아두는것도 좋을 것 같아요.
관련링크 첨부하겠습니다!
https://i-colours-u.tistory.com/22

Copy link
Member Author

Choose a reason for hiding this comment

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

캬.. 여기서 그분의.. 블로그라니 좋습니다.. 이준이 영어로 쓰죠? 폰 ㅋㅋㅋㅋ

Copy link
Member Author

Choose a reason for hiding this comment

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

이 작업도 추후에 해보자구용

@hyun99999 hyun99999 merged commit 5e8e52f into develop Jan 31, 2023
@hyun99999 hyun99999 deleted the feature/#353 branch January 31, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlwns33 dlwns33 dlwns33 approved these changes

Assignees

@hyun99999 hyun99999

Labels
Fix 코드 수정 및 버그, 오류 해결 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix: 기본 명함 앞면 수정
2 participants
@hyun99999 @dlwns33