Skip to content

[Refactor] 네이티브 SDK 로그인 구현 및 브릿지 이관#187

Merged
yummjin merged 5 commits intodevelopfrom
refactor/186
Apr 12, 2026
Merged

[Refactor] 네이티브 SDK 로그인 구현 및 브릿지 이관#187
yummjin merged 5 commits intodevelopfrom
refactor/186

Conversation

@yummjin
Copy link
Copy Markdown
Member

@yummjin yummjin commented Apr 10, 2026

🛠️ 변경 사항

  • 기능 추가 (Feature)
  • 리팩토링 (Refactor)

세부 변경 내용

  • 웹에서 로그인 관련 로직을 브릿지로 이관
  • 카카오/애플 로그인 네이티브 SDK 연동 (performKakaoLogin, performAppleLogin)
  • 브릿지 타입 정의 추가 및 authStorage 구현
  • app.config.ts에 카카오 SDK 플러그인 설정 추가
  • apiTypes.ts 타입 정의 확장

🔍 관련 이슈


📸 스크린샷 / GIF (선택)

UI 변경 없음


⚠️ 주의 사항 / 리뷰 포인트

  • 네이티브 SDK 로그인 흐름이 기존 웹 방식과 다르게 브릿지를 통해 처리되므로 웹↔네이티브 메시지 흐름 확인 필요
  • authStorage가 토큰을 저장하는 방식 검토

🔄 연관 작업

@yummjin yummjin changed the title ♻️ [refactor] 네이티브 SDK 로그인 구현 및 브릿지 이관 (#186) [Refactor] 네이티브 SDK 로그인 구현 및 브릿지 이관 Apr 10, 2026
@yummjin yummjin requested a review from seseoju April 10, 2026 15:47
@yummjin yummjin self-assigned this Apr 10, 2026
@yummjin yummjin added ✨ Feature 신규 기능을 추가합니다. 🔄 Refactor 코드 리팩토링을 진행합니다. 🔍 QA QA 사항을 적용합니다. 🔧 Improvement 현재 발생되는 문제점을 개선합니다 labels Apr 10, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request implements native social login for Kakao and Apple by establishing a bridge between the mobile app and the web view. It introduces expo-apple-authentication, @react-native-kakao/user, and expo-secure-store for secure token management. The changes include new mobile API handlers, bridge methods for authentication, and refactoring the web application's login logic to leverage native SDKs. Review feedback highlights the need for stricter environment variable validation, correction of React Hook dependency arrays to prevent closure issues, and the removal of a duplicate enum value in the API type definitions.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

우선순위가 높은 작업은 아니지만, app.config.ts에서 app.json 오버라이딩 대신 app.config.ts로 합치고 app.json 제거해도 좋을거 같아요!

Copy link
Copy Markdown
Member

@seseoju seseoju left a comment

Choose a reason for hiding this comment

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

네이티브 로그인으로 전환되면서 웹 환경에서 로그인 하는 경우에는 기존처럼 로그인하도록 추가적으로 처리도 해야 할것 같네요~! 고생하셨습니다!!👏

@yummjin yummjin merged commit e0725d8 into develop Apr 12, 2026
1 of 2 checks passed
@yummjin yummjin deleted the refactor/186 branch April 12, 2026 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 신규 기능을 추가합니다. 🔧 Improvement 현재 발생되는 문제점을 개선합니다 🔍 QA QA 사항을 적용합니다. 🔄 Refactor 코드 리팩토링을 진행합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] 로그인 로직 네이티브로 변경

2 participants