-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
카카오톡 공유, URL 공유 기능 구현 #107
Conversation
case let .공유받은_카테고리_조회(categoryId, _): | ||
return "/share/\(categoryId)" | ||
case .공유받은_카테고리_저장: | ||
return "/share" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Util Constants보고 수정
case let .포킷_오류_핸들링(baseError): | ||
switch baseError { | ||
case let .CA_001(message): | ||
state.pokitNameTextInpuState = .error(message: message) | ||
return .none | ||
default: return .none | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저는 이 과정이 필요없다고 생각함
곧바로 state.pokitNameTextInputState = .error(message: baseError.message) 하지 않고 케이스처리한 이유가???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
생각해보니 그러네여 왜 그랬지..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생많으셨습니다! 궁금한거나 이상한거 코맨트 달아놨읍니다.
- 모델 객체만들 때 init많이 사용하시는 것으로 보이는데 이러면 사용되는 모델이 어떤 이름을 가진 객체를 init하는지 알 수 없어서 저는 직관적이지 않다고 생각합니다.
let request = Model(dosomething: )
형식으로 통일해주시면 좋을 것 같아요
2.sheet의 핸들링이 필요하여 ShareLink를 사용하지 못함
이부분에 대해 명확한 근거가 필요할 것 같아요.
구현하지 못한 것
인지 못하는 상황
인지 알수있게 공식문서를 참고해서 말씀해주시면 좋을 것 같습니다.
|
434e8ca
to
b26fa40
Compare
#️⃣연관된 이슈
📝작업 내용
KakaoSDK
의존성 설치KakaoShareClient
작성URL Scheme
에 따른 공유 플로우 핸들링BaseError
작성스크린샷 (선택)
ScreenRecording_08-26-2024.01-25-21_1.MOV
ScreenRecording_08-26-2024.2.MOV
ScreenRecording_08-26-2024.01-26-10_1.MP4
💬리뷰 요구사항(선택)
CA_XXX
로 바꾸기로 결정하였고, 작성해주신BaseError
에 반영하였습니다.CategoryDetailView
와 공유된 카테고리 미리보기 화면이 유사하여,FeatureCategoryDetail
에서 공유하기 기능을 추가하는 방식으로 가려고 했으나, 도메인의 차이도 크고, 자칫CategoryDetailFeature
가 너무 비대해질 것 같아FeatureSharing
이라는 기능 모듈로 빼서 구현을 하였습니다.SwiftUI
의ShareLink
를 쓰려고 했으나, 공유 완료 시점의sheet
핸들링이 필요하여,UIKit
의UIActivityViewController
를 사용하여 별도의 공유 시트와completion
을 사용하는 방향으로 구현하였습니다.close #78