[GMSS-169] 구성 변경 시 Google Credential 로그인 요청 유지 및 login api 첫 방문 여부 응답값 추가 - #23
Open
seunghee17 wants to merge 8 commits into
Open
[GMSS-169] 구성 변경 시 Google Credential 로그인 요청 유지 및 login api 첫 방문 여부 응답값 추가#23seunghee17 wants to merge 8 commits into
seunghee17 wants to merge 8 commits into
Conversation
- Google Credential 요청 구성변경시 취소 이슈 해결 - data, domain 모듈의 인증관련 파일 기능별 분류 위한 디렉토리 이동 # Conflicts: # data/src/main/java/com/gamss/android/data/di/RepositoryModule.kt # domain/src/main/kotlin/com/gamss/android/domain/usecase/SecessionUseCase.kt
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
seunghee17
marked this pull request as ready for review
August 10, 2026 13:03
| @SerialName("refreshToken") | ||
| val refreshToken: String, | ||
| @SerialName("isFirstLogin") | ||
| val isFirstLogin: Boolean |
Collaborator
There was a problem hiding this comment.
토큰 재발급엔 얘가 없어서 missingField 날 수도 있을 것 같아요. 공유하는 모델이니 def 값을 넣어 두는 것이 좋을 것 같습니다.
| @@ -1,13 +1,14 @@ | |||
| package com.gamss.android.data.repository | |||
Collaborator
There was a problem hiding this comment.
테스트만 디렉토리가 옮겨지지 않은 것으로 보입니다. 👀
| @@ -1,9 +1,9 @@ | |||
| package com.gamss.android.data.repository | |||
|
|
|||
Collaborator
There was a problem hiding this comment.
흠.. 이친구는 어디 두는게 좋을까요?
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
3 participants
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.
작업 개요
Google Credential 요청 중 구성 변경이 발생하면 로그인 요청이 취소·유실되는 문제를 개선했습니다.
작업 유형
변경 사항
로그인 버튼 클릭 시 ViewModel에 Credential 요청 의도를 저장하도록 변경
googleSignInRequestId가 유지되는 동안LoginScreen의LaunchedEffect에서 Credential 요청 실행성공·취소·실패 시 pending 요청 상태를 명시적으로 초기화
구성 변경으로 화면이 재생성되어도 ViewModel에 남아 있는 요청을 다시 실행하도록 처리
Google Credential Launcher 책임 정리
launch()를 suspend 함수로 변경GoogleCredentialResult로 반환로그인 결과 모델 확장
isFirstLogin응답을LoginResult로 domain 계층까지 전달인증 패키지 구조 정리
AuthRepositoryImpl을data.auth패키지로 이동domain.auth패키지로 통합기존 토큰 재발급 구조 유지
TokenAuthenticator의 동기 재발급 및 동시 401 단일 재발급 구조 유지관련 이슈
관련 작업 (Notion)
스크린샷 / 동작 화면
체크리스트
develop으로 설정되어 있다feat:,fix:등)을 따른다리뷰 요청 사항
LoginScreen에서 정상적으로 다시 실행되는지 확인 부탁드립니다.googleSignInRequestId가 모두 초기화되는지 확인 부탁드립니다.auth패키지로 이동한 범위와 참조 경로가 적절한지 확인 부탁드립니다.isFirstLogin은 현재 domain까지 전달하며, 실제 온보딩 화면 이동은 후속 작업으로 남겨두었습니다.