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.

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

Merged
merged 12 commits into from
Feb 4, 2025
Merged

Conversation

syss220211
Copy link
Member

@syss220211 syss220211 commented Jan 31, 2025

📌 What is the PR?

  • 온보딩에서 postSignIn 구현 완료

🪄 Changes

  • 각 소셜 로그인으로 accessToken 을 받아온 뒤, PostSignIn을 진행하여 결과에 따라 처리할 수 있도록 구현해놓았습니다.

🙆🏻 To Reviewers

  • 네비게이션 작업 완료되면 데이터 이어보겠습니다..

💭 Related Issues

Copy link
Contributor

@FpRaArNkK FpRaArNkK left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다! 코멘트 한 번 확인부탁드려요

TnT/.gitignore Outdated
Comment on lines 81 to 82
# **/*.plist
# Support/*.plist
**/*.plist
Resources/GoogleService-Info.plist
Copy link
Contributor

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만 남겨주시는게 좋아보여요!

Comment on lines 41 to 43
public func kakaoLogout() async {
// 미구현
}
Copy link
Contributor

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()
Copy link
Contributor

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()
Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

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

요거는 빼주셔도 될 것 같아요!

Comment on lines 68 to 76
// 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)
}

Copy link
Contributor

Choose a reason for hiding this comment

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

extension DefaultUseUseCase: UseRepository 로
분리해서 작성해보는건 어떨까요??

@FpRaArNkK FpRaArNkK assigned FpRaArNkK and syss220211 and unassigned FpRaArNkK Feb 2, 2025
@FpRaArNkK FpRaArNkK added the ✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등) label Feb 2, 2025
Copy link
Contributor

@FpRaArNkK FpRaArNkK left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다! 머지 레스고 🔥🔥

@syss220211 syss220211 merged commit 725b3b4 into develop Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TNT-209] 트레이너 마이페이지 작성
2 participants