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

인증 관련 인터페이스 분리 및 애플 서버 연동 #563

Merged
merged 12 commits into from
Sep 26, 2024

Conversation

Mingyum-Kim
Copy link
Contributor

PR의 목적이 무엇인가요?

  • 소셜 로그인 종류가 늘어남에 따라 인증 관련 클래스를 인터페이스로 분리하여 유연하게 추가할 수 있도록 함.
  • 애플 서버에게 토큰 요청을 보내는 코드를 작성함.

이슈 ID는 무엇인가요?

설명

아직 개발자 계정이 없어 애플 서버에게 실제로 요청을 보내는 테스트는 작성하지 못했어요. 😂 그렇지만 인터페이스 분리한 것은 미리 올려놓으면 좋을 것 같아 PR 보냅니다.
애플 관련 부분도 우선 올리고 추후 테스트 후에 보완해서 다시 올릴게요 !

질문 혹은 공유 사항 (Optional)

@Mingyum-Kim Mingyum-Kim added BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Sep 24, 2024
@Mingyum-Kim Mingyum-Kim self-assigned this Sep 24, 2024
Copy link
Contributor

@hoyeonyy hoyeonyy left a comment

Choose a reason for hiding this comment

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

구글까지 부탁 드립니다

@Mingyum-Kim Mingyum-Kim merged commit c829def into develop-backend Sep 26, 2024
1 check passed
@Mingyum-Kim Mingyum-Kim deleted the feature/#560 branch September 26, 2024 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants