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

cd build setting #120

Merged
merged 44 commits into from
Aug 30, 2023
Merged

cd build setting #120

merged 44 commits into from
Aug 30, 2023

Conversation

likppi10
Copy link
Collaborator

No description provided.

likppi10 and others added 13 commits August 28, 2023 02:29
* uiState 변경 방식 update 로 통일

* TODO 제거

이거 예전에 지웠는데 왜 남아있지

* Ktor 를 Retrofit 으로 migration

* GuestLogin 관련 함수를 BandalartService 내에서 분리

GuestLoginService interface 로 따로 선언

* style check success

* 사용하지 않는 클래스 주석처리

* GuestLoginService Provider 추가

* KtorClient, RetrofitClient 를 별도의 networkModule 로 분리

networkModule 을 생성함에 따라 모듈의 순환 구조를 방지하기 위해 datastore 모듈 추가

* style check success

* style check success

* fix run build fail

* 스플래시 뷰모델 api 호출 이후 0.5 초 이내에 응답을 받을 경우 Loading Screen 을 띄우지 않는 로직 적용

update 로 uiState 변경 로직 통일

* chore: 코드 정리

viewModel 에서 수집한 error 는 이미 toAlertMessage 함수에 의해 custom 된 exception message 이므로 로그를 확인할 필요가 없으므로 제거

* 하드코딩된 텍스트 stringResource 로 변경
@likppi10 likppi10 added the hotfix tasks related to fix bugs label Aug 30, 2023
@likppi10 likppi10 self-assigned this Aug 30, 2023
@likppi10 likppi10 merged commit c2d6928 into main Aug 30, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix tasks related to fix bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants