diff --git a/app/src/main/java/kr/co/vividnext/sodalive/live/LiveFragment.kt b/app/src/main/java/kr/co/vividnext/sodalive/live/LiveFragment.kt index 44a55c8f..33546bfd 100644 --- a/app/src/main/java/kr/co/vividnext/sodalive/live/LiveFragment.kt +++ b/app/src/main/java/kr/co/vividnext/sodalive/live/LiveFragment.kt @@ -929,7 +929,7 @@ class LiveFragment : BaseFragment(FragmentLiveBinding::infl } viewModel.getRoomDetail(roomId) { - if (it.channelName != null) { + if (!it.channelName.isNullOrBlank()) { if (it.manager.id == SharedPreferenceManager.userId) { handler.postDelayed({ viewModel.enterRoom(roomId, onEnterRoomSuccess) diff --git a/app/src/main/java/kr/co/vividnext/sodalive/main/DeepLinkActivity.kt b/app/src/main/java/kr/co/vividnext/sodalive/main/DeepLinkActivity.kt index f180ed54..21f8c876 100644 --- a/app/src/main/java/kr/co/vividnext/sodalive/main/DeepLinkActivity.kt +++ b/app/src/main/java/kr/co/vividnext/sodalive/main/DeepLinkActivity.kt @@ -231,11 +231,7 @@ class DeepLinkActivity : AppCompatActivity() { when { roomId != null && roomId > 0 -> { - startActivity( - Intent(applicationContext, LiveRoomActivity::class.java).apply { - putExtra(Constants.EXTRA_ROOM_ID, roomId) - } - ) + routeLiveInMain(roomId) return true } @@ -332,11 +328,7 @@ class DeepLinkActivity : AppCompatActivity() { return false } - startActivity( - Intent(applicationContext, LiveRoomActivity::class.java).apply { - putExtra(Constants.EXTRA_ROOM_ID, deepLinkValueId) - } - ) + routeLiveInMain(deepLinkValueId) true } @@ -424,4 +416,19 @@ class DeepLinkActivity : AppCompatActivity() { } } } + + private fun routeLiveInMain(roomId: Long) { + val extras = Bundle().apply { + putString("room_id", roomId.toString()) + putString("deep_link_value", "live") + putString("deep_link_sub5", roomId.toString()) + } + + startActivity( + Intent(applicationContext, MainActivity::class.java).apply { + addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP or Intent.FLAG_ACTIVITY_SINGLE_TOP) + putExtra(Constants.EXTRA_DATA, extras) + } + ) + } } diff --git a/app/src/main/java/kr/co/vividnext/sodalive/main/MainActivity.kt b/app/src/main/java/kr/co/vividnext/sodalive/main/MainActivity.kt index 03fe01ca..3f202af3 100644 --- a/app/src/main/java/kr/co/vividnext/sodalive/main/MainActivity.kt +++ b/app/src/main/java/kr/co/vividnext/sodalive/main/MainActivity.kt @@ -317,11 +317,7 @@ class MainActivity : BaseActivity(ActivityMainBinding::infl val deepLinkUrl = bundle.getString("deep_link") if (!deepLinkUrl.isNullOrBlank()) { val deepLinkBundle = buildBundleFromDeepLinkUrl(deepLinkUrl) - if (deepLinkBundle != null) { - return executeBundleRoute(deepLinkBundle) - } - - return false + return executeBundleRoute(deepLinkBundle ?: bundle) } return executeBundleRoute(bundle) @@ -340,18 +336,12 @@ class MainActivity : BaseActivity(ActivityMainBinding::infl ?: bundle.getLong(Constants.EXTRA_AUDITION_ID).takeIf { it > 0 } val communityCreatorId = bundle.getString(Constants.EXTRA_COMMUNITY_CREATOR_ID)?.toLongOrNull() ?: bundle.getLong(Constants.EXTRA_COMMUNITY_CREATOR_ID).takeIf { it > 0 } - val isLiveReservation = bundle.getBoolean(Constants.EXTRA_LIVE_RESERVATION_RESPONSE) - when { roomId != null && roomId > 0 -> { viewModel.clickTab(MainViewModel.CurrentTab.LIVE) handler.postDelayed({ - if (isLiveReservation) { - liveFragment.reservationRoom(roomId) - } else { - liveFragment.enterLiveRoom(roomId) - } + liveFragment.enterLiveRoom(roomId) }, 500) return true } diff --git a/docs/20260313_예약라이브알림딥링크분기수정.md b/docs/20260313_예약라이브알림딥링크분기수정.md new file mode 100644 index 00000000..89689a86 --- /dev/null +++ b/docs/20260313_예약라이브알림딥링크분기수정.md @@ -0,0 +1,40 @@ +# 20260313 예약라이브알림딥링크분기수정 + +## 작업 체크리스트 +- [x] 알림 리스트/푸시/딥링크 라이브 라우팅 경로에서 예약 라이브 분기 위치를 정리한다. +- [x] 예약 라이브는 라이브 상세 페이지, 진행 중 라이브는 즉시 입장하도록 분기 로직을 반영한다. +- [x] 라이브룸 입장 상태에서 푸시/딥링크 실행 시 이동 확인 팝업 로직이 동일하게 동작하는지 점검하고 보정한다. +- [x] 관련 소스 LSP 진단, 단위 테스트, 빌드로 검증한다. +- [x] 검증 기록을 본 문서 하단에 누적한다. + +## 검증 기록 +- 무엇: 예약 라이브 딥링크가 `LiveRoomActivity`로 직행해 크래시를 유발하는 경로를 우회하고, `MainActivity -> LiveFragment.enterLiveRoom` 경로로 통일해 예약/진행중 상태를 내부 상세 조회로 판별하도록 수정했다. + 왜: 예약 라이브는 라이브 상세가 열려야 하고, 진행중 라이브만 즉시 입장해야 하기 때문이다. + 어떻게: `DeepLinkActivity`의 foreground 라우팅(`room_id`, `deep_link_value=live`)을 `MainActivity` 전달 방식으로 변경하고, `MainActivity.executeBundleRoute`에서 room 진입을 `enterLiveRoom` 단일 경로로 정리했다. + +- 무엇: 라이브룸 화면에서 푸시/딥링크 실행 시 이동 확인 팝업 흐름 유지 여부를 점검했다. + 왜: 라이브 입장 중에는 즉시 화면 전환 대신 사용자 확인이 필요하다. + 어떻게: `LiveRoomActivity`의 `ACTION_LIVE_ROOM_DEEPLINK_CONFIRM` 수신 -> `showDeepLinkNavigationDialog` -> `MainActivity` 전달 흐름이 그대로 유지되는지 코드 경로를 확인했다. + +- 무엇: 정적 진단/테스트/빌드를 수행했다. + 왜: 변경으로 인한 회귀 여부를 확인하기 위해서다. + 어떻게/결과: + - `lsp_diagnostics` (`DeepLinkActivity.kt`, `MainActivity.kt`): Kotlin LSP 서버 미설정으로 도구 실행 불가(환경 제약). + - `./gradlew :app:testDebugUnitTest`: 성공. + - `./gradlew :app:assembleDebug`: 성공. + +- 무엇: Oracle 리뷰에서 발견된 라우팅 누락 가능성/오분기 가능성을 보완했다. + 왜: `deep_link`가 있는 경우 원본 번들 fallback 누락과 `channelName` 빈 문자열 케이스가 실제 알림 라우팅 누락/오분기를 만들 수 있기 때문이다. + 어떻게: `MainActivity.executeBundleDeeplink`에서 `deepLinkBundle ?: bundle` fallback으로 수정했고, `LiveFragment.enterLiveRoom` 분기 조건을 `channelName.isNullOrBlank()` 기준으로 정리했다. + +- 무엇: 디바이스 수동 검증(ADB)으로 foreground 딥링크 경로를 확인했다. + 왜: 변경된 foreground 경로가 `LiveRoomActivity` 직행이 아닌 `MainActivity` 경유로 동작하는지 확인하기 위해서다. + 어떻게/결과: + - `./gradlew :app:installDebug`: 성공(실기기 설치 완료). + - `adb shell monkey -p kr.co.vividnext.sodalive.debug -c android.intent.category.LAUNCHER 1`: 앱 실행 성공. + - `adb shell am start -W -n kr.co.vividnext.sodalive.debug/kr.co.vividnext.sodalive.main.DeepLinkActivity -a android.intent.action.VIEW -d "voiceon://live/1"`: `Activity: ...MainActivity` 확인. + - `adb shell dumpsys activity activities | grep mResumedActivity`: `...MainActivity` 확인. + +- 무엇: 라이브룸 입장 중 확인 팝업 로직은 코드 경로로 재검증했다. + 왜: 로컬브로드캐스트 기반이라 ADB로 직접 이벤트 주입이 어렵기 때문이다. + 어떻게/결과: `DeepLinkActivity`의 `ACTION_LIVE_ROOM_DEEPLINK_CONFIRM` 송신과 `LiveRoomActivity`의 다이얼로그 후 `MainActivity` 이동 경로가 변경 없이 유지됨을 확인했다.