Conversation
6 tasks
seungjunGong
approved these changes
Apr 18, 2026
| val onboardingUiState by viewModel.onboardingPokeUserListUiState.collectAsStateWithLifecycle() | ||
| val checkNewInPokeState by viewModel.checkNewInPokeOnboardingState.collectAsStateWithLifecycle() | ||
| var showErrorDialog by remember { mutableStateOf(false) } | ||
| var isInitialRoutingDone by remember { mutableStateOf(false) } |
Member
There was a problem hiding this comment.
p3: 한가지 궁금한점이 있어요. remember 로 설정하게 되면 화면 회전시에 바텀시트가 다시 뜰 수도 있을거 같은데 해당 상태가 바텀시트를 띄우는 것의 방지 목적이라고 한다면 rememberSavable 로 저장 하는건 과도한 설계일까요?
Member
Author
There was a problem hiding this comment.
좋은 의견 감사합니다!
말씀해주신대로 서버에서 받아오는 isNew필드값과 바텀시트(온보딩) 상태값을 통해서 초기진입을 구분해주고 있는데요,
온보딩 초기화면이 OnboardingScreen 내부에서 bottomsheet 로 띄워주고 있기 떄문에 화면전환과 같은 configuration change 에 대해 ui를 다시 그릴때 의도치 않은 화면 이동이 일어날 것 같네요 ㅜㅜ. rememberSavable 로 설계해보겠습니다!
sonms
approved these changes
Apr 19, 2026
Comment on lines
+66
to
75
| fun checkNewInPokeOnboarding() { | ||
| viewModelScope.launch { | ||
| _checkNewInPokeOnboardingState.emit(checkNewInPokeOnboardingUseCase.invoke()) | ||
| val isNew = checkNewInPokeOnboardingUseCase.invoke() | ||
| _checkNewInPokeOnboardingState.value = isNew | ||
|
|
||
| if (isNew) { | ||
| updateNewInPokeOnboarding() | ||
| } | ||
| } | ||
| } |
Member
There was a problem hiding this comment.
p1: usecase에 operator fun 이 정의되어 있다면.invoke를 생략하는 것이 가독성도 올라가고 안드로이드 컨벤션 상으로도 단일 행동을 클래스로 캡슐화하고, 사용할 때는 함수처럼 호출한다 입니당 지켜주세요~
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
5 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.
Related issue 🛠
Work Description ✏️
누가 나를 찔렀어요에서 이미지, 이름은 안 뜨고 보낸 문구만 뜨는 오류
콕찌르기 모든 ApiError시 오늘 콕 찌르기를 다 썼어요 라고 안드 자체 메세지 발행
10번 찌르기 시 제한되는 이슈 해결 (콕찌르기 횟수 제한 제거)
=> 해당 부분들은 승준님과 겹치는 부분이라 대승준님께서 작업해주셨습니다.
(사용자가 콕찌르기 온보딩 뷰에 최초 진입 > 사용자가 온보딩 뷰를 이탈하고 이후 다시 콕찌르기 화면에 진입 > 온보딩 뷰에서의 콕찌르기 진행 여부에 상관없이 사용자는 바로 메인 화면 진입 가능)
Screenshot 📸
Uncompleted Tasks 😅
To Reviewers 📢