-
Notifications
You must be signed in to change notification settings - Fork 0
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
[TNT-158] postSingIn 구현 완료 #44
Conversation
…158-trainerSignup
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.
고생많으셨습니다! 코멘트 한 번 확인부탁드려요
TnT/.gitignore
Outdated
# **/*.plist | ||
# Support/*.plist | ||
**/*.plist | ||
Resources/GoogleService-Info.plist |
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.
저러면 Config/Info.plist도 ignoring이라 **/GoogleService-Info.plist
만 남겨주시는게 좋아보여요!
public func kakaoLogout() async { | ||
// 미구현 | ||
} |
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.
미구현은 주석이나 삭제처리해주시면 좋을 것 같아요!
} | ||
|
||
// public func appleLoginResult() async throws -> Result<PostSocialLoginReqDTO, Error> { | ||
// let result = await loginManager.appleLogin() |
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.
불필요한 주석은 삭제해주세요!
@@ -13,7 +13,7 @@ import Domain | |||
|
|||
/// 사용자 관련 네트워크 요청을 처리하는 UserRepository 구현체 | |||
public struct UserRepositoryImpl: UserRepository { | |||
|
|||
private let loginManager = SNSLoginManager() |
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.
여기에 추가되는 이유가 있을까요??
@@ -7,6 +7,7 @@ | |||
// | |||
|
|||
import Dependencies | |||
import SwiftUI |
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.
요거는 빼주셔도 될 것 같아요!
// MARK: - Repository | ||
public func postSocialLogin(_ reqDTO: PostSocialLoginReqDTO) async throws -> PostSocialLoginResDTO { | ||
return try await userRepostiory.postSocialLogin(reqDTO) | ||
} | ||
|
||
public func postSignUp(_ reqDTO: PostSignUpReqDTO, profileImage: Data?) async throws -> PostSignUpResDTO { | ||
return try await userRepostiory.postSignUp(reqDTO, profileImage: profileImage) | ||
} | ||
|
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.
extension DefaultUseUseCase: UseRepository 로
분리해서 작성해보는건 어떨까요??
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.
고생많으셨습니다! 머지 레스고 🔥🔥
📌 What is the PR?
🪄 Changes
🙆🏻 To Reviewers
💭 Related Issues