Skip to content

[feat]: deeplink 리팩토링 firebase 의존성 문제 해결#55

Merged
Roy-wonji merged 5 commits intomainfrom
refaotor/deeplink
Dec 12, 2025
Merged

[feat]: deeplink 리팩토링 firebase 의존성 문제 해결#55
Roy-wonji merged 5 commits intomainfrom
refaotor/deeplink

Conversation

@Roy-wonji
Copy link
Contributor

✨ 작업 내용

  • Domain에 딥링크 목적지/결과 모델 추가 및 파서(DeeplinkRouter) 구현
  • MainCoordinator 딥링크 처리 로직을 DeeplinkRouter 기반으로 리팩터링하고 네비게이션 헬퍼 분리
  • Tuist 패키지 설정에 GoogleSignIn/AppAuth/GTMAppAuth 프레임워크 지정, Firebase SDK 버전 12.7.0으로 업데이트

📝 참고 사항

  • 별도 테스트 실행하지 않음
  • 초대 코드 딥링크는 로그인 필요 시 delegate로 전달하도록 분기

Motivation 🥳 (코드를 추가/변경하게 된 이유)

  • 딥링크 파싱을 공용 라우터로 분리해 재사용성과 테스트 용이성 향상
  • 딥링크 이동 경로별 네비게이션을 명시적으로 관리해 유지보수성 개선
  • 인증 연동 라이브러리와 Firebase 버전 업데이트

Key Changes 🔥 (주요 구현/변경 사항)

  • DeeplinkDestination/TravelDeeplink/DeeplinkResult 모델 정의
  • DeeplinkRouter로 travel/detail/settings/expense/settlement, invite 코드 파싱 추가
  • MainCoordinator 딥링크 처리 → 목적지별 라우팅/로그인 분기 헬퍼화
  • Tuist 패키지: GoogleSignIn/AppAuth/GTMAppAuth 프레임워크 지정 및 Firebase 12.7.0 적용

To Reviewers 🙏 (리뷰어에게 전달하고 싶은 말)

  • DeeplinkRouter 파싱 규칙과 네비게이션 헬퍼 흐름 확인 부탁드립니다.
  • Firebase 버전 업에 따른 빌드 영향 여부 체크해 주세요.

Copy link
Contributor

@minneee minneee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Contributor

@Peter1119 Peter1119 left a comment

Choose a reason for hiding this comment

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

훨씬 깔끔해졌네요 ~! 고생하셨습니다 ~!

@Roy-wonji Roy-wonji merged commit fcd336a into main Dec 12, 2025
@Roy-wonji Roy-wonji deleted the refaotor/deeplink branch December 12, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants