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

Refactor/#874 network view model 리팩터링 #875

Merged
merged 5 commits into from
Dec 13, 2023

Conversation

ki960213
Copy link
Collaborator

#️⃣ 연관된 이슈

close : #874

📝 작업 내용

  • CommonUiEvent 클래스명을 NetworkUiEvent로 변경
  • ScreenUiState 클래스명을 NetworkUiState로 변경
  • 위 클래스와 관련된 기능을 RefreshableViewModel에서 NetworkViewModel로 옮기기
  • MessageList 화면에 NetworkErrorView와 ProgressBar 추가
  • RecruitmentDetailViewModel 불필요한 줄바꿈 제거

스크린샷 (선택)

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 10분
실제 소요 시간 : 20분

💬 리뷰어 요구사항 (선택)

간단한 코드이므로 바로 머지하겠습니다.

@ki960213 ki960213 added Android 안드로이드 관련 이슈 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경 High Priority 리뷰 우선순위가 높은 PR labels Dec 13, 2023
@ki960213 ki960213 self-assigned this Dec 13, 2023
@ki960213 ki960213 merged commit 16efcf6 into android-main Dec 13, 2023
1 check passed
@ki960213 ki960213 deleted the Refactor/#874-NetworkViewModel_리팩터링 branch December 13, 2023 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant