-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feat/#81] splash api 연결 및 tendency result api / UI 작업 #90
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
API 연결하느라 수고하셨습니다!!!!!!
when (state) { | ||
AuthState.LOADING -> return@onEach | ||
AuthState.SUCCESS -> navigateToDashBoardScreen() | ||
AuthState.FAILURE -> navigateToSignInScreen() | ||
AuthState.SIGNUP -> return@onEach | ||
AuthState.SIGNIN -> return@onEach | ||
AuthState.TENDENCY -> navigateToTendencyScreen() | ||
AuthState.EMPTY -> return@onEach |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호 state 상태가 많군요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
바람직합니다.
override suspend fun getSplash(): Result<Unit> = kotlin.runCatching { | ||
authDataSource.getSplash() | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코틀린을 사랑하는 마음이 담겨있네요
) : ProfileRepository { | ||
override suspend fun getUserProfile(): Result<UserProfileRequestModel> = runCatching { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
줄바꿈이 좋을 것 같아요 ~
) : ViewModel() { | ||
private val _userState = MutableStateFlow(AuthState.LOADING) | ||
val userState: StateFlow<AuthState> = _userState | ||
fun getHasAccessToken(): Boolean = tokenRepository.getAccessToken().isNotBlank() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
줄이..아쉽네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리얼 갓동민 코멘트 달 게 업네욥
최고십니다~!!!!~!
enum class AuthState { | ||
LOADING, SUCCESS, FAILURE, SIGNUP, SIGNIN, TENDENCY, EMPTY | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오@!@! signup, signin, tendency도 있군요!
⛳️ Work Description
📸 Screenshot
📢 To Reviewers