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 수정 (#399) #400

Merged
merged 9 commits into from
Apr 11, 2023
Merged

fix: 명함 리스트 재정렬 API 수정 (#399) #400

merged 9 commits into from
Apr 11, 2023

Conversation

hyun99999
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 명함 리스트 재정렬 API 를 수정하였습니다.
  • CardListEditRequest 에서 CardReorderInfo 로 데이터 모델을 변경하였습니다.
  • judgeStatus 에 타입을 넘겨서 하나로 통일했습니다.

🚨 참고사항

  • 민재쿤이 리스트를 옮길때마다 서버통신을 하고 있었고, 이게 더 좋은 방법이라고 생각이 들어서 기존에 뒤로가기를 통해 한꺼번에 바꾸는 것에서 계획을 수정했습니당
  • swift 5.8, Xcode 14.3 부터 탈출 클로저 안에서 [weak self] 을 사용하고 옵셔널 바인딩을 사용하면 self 를 사용하지않아도 된다고 합니다! 적용해보았습니다.

📮 관련 이슈

@hyun99999
add: 업데이트 내용 조회 / 사용자 업데이트 정보 조회 API 를 위한 파일 생성 (TeamNADA#397)
6763fcb
@hyun99999
feat: 업데이트 내용 조회 / 사용자 업데이트 정보 조회 API 데이터 모델 구현 (TeamNADA#397)
c8b8d6f
@hyun99999
feat: Service 구현 (TeamNADA#397)
666e764
@hyun99999
feat: API 구현 (TeamNADA#397)
ddb88f3 
- final class 로 구현
    - 상속할 수 없도록 구현하여 static dispatch 로 실행되도록 설정.
- judgeStatus() 메서드에서 타입을 정할 수 있도록해서 타입별로 여러개 생성하던 것에서 하나의 메서드를 사용하도록 리펙토링.
@hyun99999
fix: CardListEditRequest 데이터 모델을 수정 (TeamNADA#399)
ec41cd8 
- CardReorderInfo 로 수정.
@hyun99999
fix: 명함 리스트 재정렬 Service 수정 (TeamNADA#399)
cf59ec8 
- put 에서 post 로 변경.
- 데이터 모델 변경.
@hyun99999
feat: CardAPI 수정 (TeamNADA#399)
8e24482 
- judgeStatus 메서드로 통일.
- cardReorder() 메서드 수정.
@hyun99999
fix: 명함 정렬시에 서버통신 (TeamNADA#399)
d318a8a
@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Apr 6, 2023
@hyun99999 hyun99999 requested a review from dlwns33 April 6, 2023 23:48
@hyun99999 hyun99999 self-assigned this Apr 6, 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.

명함 재정렬까지 모두 확인했습니다!
외로운 작업의 길... 수고 많으셨습니다..🙇‍♀️

@hyun99999 hyun99999 merged commit b6dbaf8 into TeamNADA:develop Apr 11, 2023
@hyun99999 hyun99999 deleted the feature/#399 branch April 11, 2023 06:02
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