Skip to content

feat: DeepLinkFactory에 딥링크 정책 문서의 전체 화면 라우트 추가 - #110

Open
S-J-Kim wants to merge 1 commit into
developfrom
feature/deeplink-factory-routes
Open

feat: DeepLinkFactory에 딥링크 정책 문서의 전체 화면 라우트 추가#110
S-J-Kim wants to merge 1 commit into
developfrom
feature/deeplink-factory-routes

Conversation

@S-J-Kim

@S-J-Kim S-J-Kim commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

PR 체크리스트

Merge 전에 아래 체크리스트가 모두 체크되었는지 확인해주세요

  • 마지막 commit전에 빌드하여 코드 스타일 점검하였나요?
  • 모든 Test 코드를 실행하여 PASS했나요?
  • Swagger 명세를 작성하였나요?
  • 최소 1명의 리뷰와 Approve를 받았나요?

PR 타입

PR의 타입을 체크해주세요

  • 버그 픽스 - 이슈 링크 :
  • 신규 기능 개발
  • 기능 수정
  • 테스트 코드 추가
  • 코드 스타일 업데이트 (formatting, 변수명 수정 등)
  • 리팩토링 (기능 수정은 없고 코드 재사용성을 위한 메서드 분리 등)
  • 설정 파일 수정
  • 기타

백로그 및 이슈 링크

  • 링크 :

변경된 사항

DeepLinkFactory에 딥링크 정책 문서의 전체 화면(screen) 라우트 반영

  • Notion 딥링크 정책 가이드에 정의된 17개 screen을 DeepLinkFactory의 static 메서드로 전부 추가 (home, feed, mypage, feedback, realtime-nag, notification, myinfo, setting, setting/myinfo, setting/notification, setting/notice, setting/notice/detail, setting/policy, setting/account, setting/badge, received-feedback, cheer-collection)
  • 기존 home(LocalDate) 시그니처는 유지해서 기존 호출부(RetrieveReceivedTaskFeedbackResult)는 변경 없이 그대로 동작
  • feedback 라우트의 tab 쿼리 파라미터(receive/send)는 임의 문자열 대신 DeepLinkFactory.FeedbackTab enum으로 타입 안전하게 받도록 함

기타 전달 사항

  • 이번 변경은 라우트 빌더(Factory)만 추가한 것으로, 아직 어떤 NotificationTemplateCode가 어떤 라우트를 쓸지 매핑하는 로직(리졸버)이나 실제 호출부 배선은 포함하지 않았습니다. 후속 작업으로 예정.
  • 테스트 코드는 추가하지 않았고, ./gradlew compileJava만 통과 확인했습니다. ./gradlew test는 이 세션에서 실행하지 않았습니다.

딥링크 정책 가이드(Notion)에 정의된 17개 screen을 DeepLinkFactory의
static 메서드로 채워, 딥링크 패턴의 관리 주체를 DB가 아닌 코드로 이동.
feedback 라우트의 tab 파라미터는 FeedbackTab enum으로 타입 안전하게 처리.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

2 participants