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: 명함 조회 API 수정 (#384) #398

Merged
merged 5 commits into from
Apr 11, 2023
Merged

fix: 명함 조회 API 수정 (#384) #398

merged 5 commits into from
Apr 11, 2023

Conversation

hyun99999
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 명함 조회 API 구현
  • 명함 페이징 조회 API 구현
  • 기본 파라미터를 사용해서 동일한 명함 조회 Service 메서드에서 분기처리하여 사용하였습니다. 좀 더 직관적이고 결합도를 고려해서 작성해보았습니다.
  • [Card] 로 대체해서 사용할 것이고, offset 은 더이상 응답에 없음. CardListLookUp 데이터 모델은 삭제하였습니다.

🚨 참고사항

  • 현재 명함 목록 조회 즉, 나의 명함 순서를 바꾸기 위해 타이틀만 받아오는 API 경우 안나와서 그냥 명함 조회를 그대로 쓸지 아니면 주실지 잘 모르겠심더! 기다려보고 시간이 남았는데 안나왔으면 한 번 여쭈어보겠습니당
  • 그래서 judgeCardListFetchStatus 메서드를 주석처리하였습니당
  • FrontViewController 에서 offSet 대신 pageNumber 을 그대로 넣어서 로직을 따르도록 하였스빈다.

📮 관련 이슈

@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Apr 3, 2023
@hyun99999 hyun99999 requested a review from dlwns33 April 3, 2023 07:10
@hyun99999 hyun99999 self-assigned this Apr 3, 2023
@hyun99999 hyun99999 changed the title fix: 명함 조회 API 수정 fix: 명함 조회 API 수정 (#384) Apr 4, 2023
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.

image

NADA-iOS-forRelease/Sources/ViewControllers/Group/QRScanViewController.swift Outdated
Comment on lines 159 to 165
let nextVC = CardResultBottomSheetViewController()
.setTitle(card.userName)
.setHeight(574)
nextVC.cardDataModel = card
nextVC.modalPresentationStyle = .overFullScreen
nextVC.status = .addWithQR
self.present(nextVC, animated: false, completion: nil)
Copy link
Member

Choose a reason for hiding this comment

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

요것도 나중에 moduleFactory 에서 관리하는것으로 변환하면 좋을 것 같아요!

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 fbd1c17 into TeamNADA:develop Apr 11, 2023
@hyun99999 hyun99999 deleted the feature/#384 branch April 11, 2023 05:43
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: 명함 조회 API 수정
2 participants
@hyun99999 @dlwns33