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

feat: Setting Flow API Connect #88

Merged
merged 15 commits into from
Aug 12, 2024
Merged

feat: Setting Flow API Connect #88

merged 15 commits into from
Aug 12, 2024

Conversation

stealmh
Copy link
Member

@stealmh stealmh commented Aug 12, 2024

#️⃣연관된 이슈

#84

📝작업 내용

설정창 관련한 API 및 화면연결 작업을 했습니다.

  • 닉네임 수정 API 연결
  • 회원탈퇴 API 연결
    공통: 구글 / 애플 로그인의 회원탈퇴 로직을 추가했습니다.
    애플 회원탈퇴를 위해 애플서버로 직접 요청하는 로직을 추가했습니다.

이를 위해 Team ID, Key ID, p8파일 등이 필요했으며, 이를 Private Repository로부터 받아옵니다. 기존과 동일하게 Secret.xcconfig파일에서 ID값들을 확인할 수 있으며, CoreKit에서도 사용이 되도록 configuration을 수정했습니다.
또한 p8파일도 CoreKit의 Resources에서 관리가 필요할 것 같아 세팅을 수정했습니다.

  • 이 외 링크로 이동되는 화면들 연결작업

스크린샷 (선택)

💬리뷰 요구사항(선택)

close #84

@stealmh stealmh self-assigned this Aug 12, 2024
@stealmh stealmh added Feat 기능구현 API 🛜 기능구현 - API labels Aug 12, 2024
@stealmh stealmh changed the title feat: 회원탈퇴 API 연결 feat: Setting Flow API Connect Aug 12, 2024
Copy link
Contributor

@ShapeKim98 ShapeKim98 left a comment

Choose a reason for hiding this comment

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

오우 쉽지 않았을텐데 너무 수고많으셨습니다..
저는 애플 서버로 회원탈퇴 요청 보내는 것을 서버가 하는것으로 알고 있었는데.. 이렇게 클라이언트 단에서도 많이들 하는지 궁금합니다..(저희는 서버분들 요청으로 이런 방향으로 맞춰지긴 했..) @stealmh @ITlearning

@@ -25,6 +25,7 @@ let package = Package(
.package(url: "https://github.com/pointfreeco/swift-composable-architecture", from: "1.10.4"),
.package(url: "https://github.com/google/GoogleSignIn-iOS", "7.0.0" ..< "7.1.0"),
.package(url: "https://github.com/Moya/Moya", from: "15.0.0"),
.package(url: "https://github.com/firebase/firebase-ios-sdk", "10.28.0" ..< "10.28.1")
.package(url: "https://github.com/firebase/firebase-ios-sdk", "10.28.0" ..< "10.28.1"),
.package(url: "https://github.com/Kitura/Swift-JWT", from: "4.0.1")
Copy link
Contributor

Choose a reason for hiding this comment

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

jwt 만들어주는 라이브러리가 있군요..

Copy link
Member Author

Choose a reason for hiding this comment

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

너무 오랜만에 만들어봐서 써드파티써버렸읍니다…

@ShapeKim98 ShapeKim98 merged commit 9eefba2 into develop Aug 12, 2024
1 check passed
@stealmh stealmh deleted the feat/#84-setting-api branch September 3, 2024 12:26
@stealmh stealmh linked an issue Oct 2, 2024 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API 🛜 기능구현 - API Feat 기능구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting Flow Connect
2 participants