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

feat: 취향 조회 API 로 명함 생성 뒷면 구성 (#393) #395

Merged
merged 17 commits into from
Apr 11, 2023
Merged

feat: 취향 조회 API 로 명함 생성 뒷면 구성 (#393) #395

merged 17 commits into from
Apr 11, 2023

Conversation

hyun99999
Copy link
Member

@hyun99999 hyun99999 commented Apr 2, 2023 •
edited
Loading

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 명함 뒷면의 이미지 에셋 추가
  • 명함 생성 및 조회를 위한 데이터 모델 수정.
    • Card 와 CardCreationRequest 모델 수정.
    • 모델이 수정되면서 로직에 사용되는 멤버 변수 혹은 위젯에서 사용되는 멤버변수들을 모두 반영해서 수정했습니다.
  • 명함 뒷면 수정(기본)
    • 명함 뒷면이 검정카드로 통일되었기 때문에 나눠져있던 init 메서드도 합치고, 리펙토링하였습니다.
  • 명함 생성 뒷면 수정(기본)
    • UITextView 를 사용해서 TMI 를 입력받고 있는데 이때 여러 시나리오에서 정상 작동하는지, 총 몇자까지 쓸 수 있는지 그래서 제한두는 기능 추가까지 고려하고 있습니다.
  • 명함 생성 API 수정(design: 명함 만들기 뷰 순서 변경 #383), 취향 조회 API 구현

🚨 참고사항

  • todo, fixme 주석을 확인 후 제거
  • 명함 배경을 로컬에서 다루는 기능은 추후에 구현될 예정이기 때문에 주석 처리했습니다.
  • 아직 서버 API 가 완성되지 않아서 서버쌤들과 고려한 내용을 미리 반영해서 구현했습니다.
  • 이후에 테스트하면서 버그나 수정사항은 추가로 이슈 만들어서 진행하겠습니다.
  • 취향 조회.. 로 뒷면 만들다가.. 이렇게 왔습니다. 데이터모델이랑 로직이 다 엮겨있어서 PR 이 커졌네요 ㅜ

📮 관련 이슈

@hyun99999
feat: 명함 뒷면에 취향 목록 전달 (TeamNADA#393)
2a7be37
@hyun99999
fix: BackCardCreationDelegate 수정 (TeamNADA#393)
ed31ffa 
- 필수 정보를 [String], 옵션 정보를 String 으로 전달.
@hyun99999
fix: BackCardDataModel 수정 (TeamNADA#393)
3feec39
@hyun99999
add: 이미지 에셋 추가 (TeamNADA#393)
ff3b145
@hyun99999
fix: Card 데이터 모델 수정 (TeamNADA#393)
a61679d
@hyun99999
fix: Card 데이터 모델 수정으로인한 MyCardWidget 수정 (TeamNADA#393)
e2eee6b
@hyun99999
fix: Card 데이터 모델 수정으로 인한 IntentHandler 수정 (TeamNADA#393)
b1c19d0
@hyun99999
fix: Card 데이터 모델 수정되면서 수정 (TeamNADA#393)
c7af9dc
@hyun99999
fix: BackCardCell 수정 (TeamNADA#393)
ba62721 
- 리펙토링 및 UI 변경으로 수정.
- initCell 메서드 수정
@hyun99999
delete: 주석 제거 (TeamNADA#393)
c7dbdd7
@hyun99999
fix: CardCreationRequest 수정 (TeamNADA#393)
61917c2 
- FrontCardDataModel 와 BackCardDataModel 수정
- delegate 파라미터 자료형도 수정
@hyun99999
comment: 추후에 명함 배경 로컬에서 다루기 때문에 주석 (TeamNADA#393)
ecba86b
@hyun99999
fix: BackCardCreationCollectionViewCell 수정 (TeamNADA#393)
9460108 
- UI 수정 및 기능 구현
@hyun99999
fix: 명함 생성 API 구현 (TeamNADA#393)
db406a7
@hyun99999
fix: BackCardCell 의 init 메서드 반영 (TeamNADA#393)
3bd5339
@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Feat 새로운 기능 구현 labels Apr 2, 2023
@hyun99999 hyun99999 requested a review from dlwns33 April 2, 2023 19:47
@hyun99999 hyun99999 self-assigned this Apr 2, 2023
@hyun99999 hyun99999 mentioned this pull request Apr 2, 2023
Closed
10 tasks
@hyun99999 hyun99999 changed the title feat: 취향 조회 API 로 명함 생성 뒷면 구성 feat: 취향 조회 API 로 명함 생성 뒷면 구성 (#393) Apr 2, 2023
@hyun99999 hyun99999 mentioned this pull request Apr 3, 2023
Closed
3 tasks
@hyun99999
delete: CardListLookUp 삭제
66318cf 
- [Card] 로 대체해서 사용할 것이기 때문에 삭제. offset 은 더이상 응답에 없음.
dlwns33
dlwns33 approved these changes Apr 11, 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.

선생님 코드 다 읽어봤습니다!
뒷면 수정을 다 해주셨네요... 수고하셨습니다

@hyun99999 hyun99999 merged commit 6771589 into TeamNADA:develop Apr 11, 2023
@hyun99999 hyun99999 deleted the feature/#393 branch April 11, 2023 05:36
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
Feat 새로운 기능 구현 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: 취향 조회 API 로 명함 생성 뒷면 구성 fix: 명함 생성 API 수정
2 participants
@hyun99999 @dlwns33