Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix/#722 에뮬레이터 splash activity에서 다음화면으로 넘어가지 못하는 버그 #723

Conversation

chws0508
Copy link
Collaborator

#️⃣연관된 이슈

#722

📝작업 내용

vesionCode 49로 업데이트
인앱 업데이트 실패 시, 다음 화면으로 이동

1줄 바꿨고, 오류가 뜨는 이유는, 앱에 Playstore가 깔려있지 않으면, 인앱 업데이트가 실패해서, OnFailure 로 넘어가는데, 이때도 다음 화면으로 넘어가게 해주어야 앱 심사가 통과할 것 같아서 이렇게 바꿨습니다.

예상 소요 시간 및 실제 소요 시간

30분/ 30분

@chws0508 chws0508 added 버그 개발자가 의도하지 않은 상황 Android 안드로이드 관련 이슈 labels Oct 16, 2023
@chws0508 chws0508 added this to the 6차 스프린트 milestone Oct 16, 2023
@chws0508 chws0508 self-assigned this Oct 16, 2023
Copy link
Member

@tmdgh1592 tmdgh1592 left a comment

Choose a reason for hiding this comment

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

@chws0508
개인적인 의견을 남겨놓았으니 확인 바랍니다.

@@ -71,7 +71,7 @@ class SplashActivity : ComponentActivity() {

appUpdateManager.appUpdateInfo.addListener(
onSuccess = { updateInfo -> updateInfo.handleUpdateInfo(isAutoLogin) },
onFailed = { showToast(R.string.all_network_error_title) },
onFailed = { navigateToNextScreen(isAutoLogin) },
Copy link
Member

Choose a reason for hiding this comment

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

개인적으로 바로 다음 화면으로 이동하면 버전 체크를 하는 이유가 없다고 생각해요.
차라리 플레이스토어로 이동하는 것은 어떨까요?

+) 기존에 navigateToAppStore() 라는 메서드도 웹 브라우저에서 실행될 수 있도록 아래 함수로 대체해주시면 좋을 것 같아요.

appUpdateManager.appUpdateInfo.addListener(
            onSuccess = { updateInfo -> updateInfo.handleUpdateInfo(isAutoLogin) },
            onFailed = {
                showToast(R.string.splash_not_installed_playstore)
                navigateToPlayStore()
            },
        )

...
private fun navigateToPlayStore() {
    val intent = Intent(Intent.ACTION_VIEW)
        .addCategory(Intent.CATEGORY_BROWSABLE)
        .setData(Uri.parse("market://details?id=$packageName"))
    startActivity(intent)
}

Copy link
Member

Choose a reason for hiding this comment

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

<string name="splash_not_installed_playstore">플레이스토어가 설치되지 않았거나 로그인하지 않았어요 😥</string>

@chws0508 chws0508 merged commit cc4f42e into android-main Oct 16, 2023
1 check passed
@amaran-th amaran-th deleted the Hotfix/#722-에뮬레이터_SplashActivity에서_다음화면으로_넘어가지_못하는_버그 branch October 19, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈 버그 개발자가 의도하지 않은 상황
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants