Conversation
DynamicLinkRepositoryImpl.createLink()가 ChottuLink의 setSelectedPath("report")를
하드코딩해서 호출하고 있었다. 그런데 이 저장소는 지인 피드백 공유 링크
생성에만 쓰이고 있고("report" 경로는 ChottuLink 대시보드에 인터뷰 리포트
공유용으로 등록된 값), 지인 피드백 딥링크에는 맞지 않는 경로라 매번
createDynamicLink() 호출이 실패했다.
CreateFeedbackShareDynamicLinkUseCase가 이 실패를 조용히 원시 딥링크
(`hilit://feedback/{token}`)로 대체하도록 설계돼 있어 사용자에게는 에러가
드러나지 않았지만, 그 raw 커스텀 스킴 링크는 카카오톡 등 공유 채널에서
탭 가능한 링크로 인식되지 않고 https 폴백도 없어 사실상 동작하지 않는
링크였다 — 이것이 "지인피드백 링크 미동작" 증상의 원인.
setSelectedPath() 호출을 제거해 도메인 기본 경로로 동적 링크를 생성하도록
수정하고, 향후 같은 방식으로 대시보드 설정이 어긋나도 조용히 묻히지
않도록 실패 시 로그를 남기도록 추가.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
hilit://feedback 커스텀 스킴 intent-filter만 등록돼 있고, ChottuLink가 발급하는 실제 공유 링크 도메인(https://hilit.chottu.link)에 대한 App Link intent-filter가 없었다. 그래서 사용자가 공유받은 https 링크를 브라우저에서 열면 OS가 이 앱으로 연결해줄 방법이 없어 항상 브라우저로 떨어졌고, ChottuLink 랜딩 페이지의 JS 리다이렉트(비동기 fetch 이후 커스텀 스킴으로 이동)에만 의존했는데 브라우저에서 신뢰성 있게 앱을 열지 못했다. https://hilit.chottu.link/.well-known/assetlinks.json 에는 이미 이 앱 패키지/서명으로 등록이 끝나 있어(대시보드 쪽은 준비돼 있었음), autoVerify App Link intent-filter를 hilit.chottu.link 도메인으로 추가하기만 하면 됐다. 기기에서 검증해보니 도메인 검증(verified) 즉시 성공하고, https 링크가 브라우저를 거치지 않고 바로 앱을 여는 것을 확인했다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
link.lastPathSegment 가 null 이 아닌 빈 문자열을 반환하는 경우까지는 막지 못해, 그런 경우 EnterGuestFeedbackUseCase 가 네트워크 호출도 없이 검증 오류로 즉시 실패하고 "앱을 종료한 뒤 링크를 다시 열어주세요" 에러 모달만 뜨는 혼란스러운 UX가 됐다. token != null 을 !token.isNullOrBlank() 로 강화해 애초에 온보딩 화면으로 진입하지 않도록 한다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Collaborator
Author
|
#202 번 파생이슈 있어 추가 생성하였습니다. |
Contributor
Catalog Preview상태: ✅ 최신 배포 성공 Preview: https://yapp-github.github.io/28th-App-Team-1-Android/pr/201/ Commit: 실행: GitHub Actions |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚩 연관 이슈
closed #175
📝 작업 내용
DynamicLinkRepositoryImpl.createLink()가 ChottuLinksetSelectedPath("report")를 하드코딩하고 있었는데, 이 저장소는 실제로 지인 피드백 공유 링크 생성에만 쓰이고 있어("report"는 인터뷰 리포트 공유용으로 대시보드에 등록된 경로) 매번 링크 생성이 실패,CreateFeedbackShareDynamicLinkUseCase가 이를 조용히 원시 딥링크(hilit://feedback/{token})로 대체하고 있었음 →setSelectedPath제거, 실패 시 로그 추가hilit.chottu.link도메인에 대한 Android App Link(autoVerify) intent-filter가 매니페스트에 없어 https 공유 링크를 브라우저에서 눌러도 앱이 열리지 않던 문제 수정 (assetlinks.json은 이미 대시보드에 등록돼 있어 매니페스트만 추가하면 됐음 — 기기에서 도메인 검증(verified) 및 브라우저 경유 없이 바로 앱 오픈되는 것 확인)MainActivity.navigateToFeedbackOnboarding에서 token이 빈 문자열인 경우까지 방어하도록!token.isNullOrBlank()로 강화 (빈 토큰이면 온보딩 화면 진입 자체를 막아, 진입 직후 에러 모달만 뜨는 혼란스러운 UX 방지)🏞️ 스크린샷 (선택)
🗣️ 리뷰 요구사항 (선택)