[Feat] #466 -회원가입 API 연동 #469
Conversation
|
| } | ||
|
|
||
| public var validationType: ValidationType { | ||
| return .none |
There was a problem hiding this comment.
이 부분 없어도 validationType 기본값은 .none 아닌가용?
| } | ||
|
|
||
| } | ||
|
|
|
|
||
| public enum SocialAPI { | ||
| case getSocialAccount(phone: String) | ||
| case changeSocialAccount(entity: ChangeSocialAccountEntity) |
There was a problem hiding this comment.
CoreAuthAPI에는 dto로 명명되어 있던데 명칭 통일하시는 거 어떠실지요!
There was a problem hiding this comment.
entity 와 dto중에 무엇으로 통일할까요!
| @@ -35,7 +37,8 @@ extension BaseAPI { | |||
| public var baseURL: URL { | |||
| var base = Config.Network.baseURL | |||
| let operationBaseURL = Config.Network.operationBaseURL | |||
|
|
|||
| let coreAuthBaseURL = Config.Network.coreAuthBaseURL | |||
There was a problem hiding this comment.
네네 맞아용
새로운 서버를 만든거라 baseURL이 달라졌습니다
| public protocol CoreAuthService { | ||
| func sendVerifyCode(_ dto: SendVerificationCodeEntity) -> AnyPublisher<Int, Error> | ||
| func verifyCode(_ dto: VerifyCodeEntity) -> AnyPublisher<BaseEntity<VerifyResultEntity>, Error> | ||
| func login(_ dto: LoginEntity) -> AnyPublisher<BaseEntity<CoreSignInEntity>, Error> | ||
| } |
There was a problem hiding this comment.
AnyPublisher<Int, Error> Error 타입을 구체화하는 것도 좋아보입니다.
발생할 수 있는 에러 모두 정의하고 그걸 기반으로 테스트 코드 짜도 좋을 것 같아요.
아니면 에러처리를 다른 곳에서 해주고 있나요?
There was a problem hiding this comment.
Repository에서 Domain별 Error로 변환해주도록 구현할 예정입니다.
There was a problem hiding this comment.
UseCase에선 해당 Error 를 catch하여 SideEffect 스트림으로 연결할 예정이구욥!
There was a problem hiding this comment.
Data 계층의 Error 타입 정의는 팀원들과 더 소통하고 정하면 좋을 것 같아욥
🌴 PR 요약
🌱 작업한 브랜치
🌱 PR Point
📌 참고 사항
📸 스크린샷
Data 계층 위주의 작업이라 스크린샷은 없습니다.
📮 관련 이슈