Skip to content

Rhedi/week10#100

Open
hajinmn-stack wants to merge 8 commits into
UMC-10th:mainfrom
hajinmn-stack:Rhedi/week10
Open

Rhedi/week10#100
hajinmn-stack wants to merge 8 commits into
UMC-10th:mainfrom
hajinmn-stack:Rhedi/week10

Conversation

@hajinmn-stack

Copy link
Copy Markdown

📌 PR 제목

해당 pr에 대한 내용을 요약해주세요

  1. 빌드된 AAB 파일 경로 pr에 업로드 하기
  • Claude Code CLI 설치하고 프로젝트 루트에서 실행하기 (다른 CLI도 괜찮습니다)
  • 본인이 작성한 코드 선택해서 코드 리뷰 요청하기
  • 리뷰 결과를 바탕으로 실제로 코드 개선해보기

#️⃣ 연관된 이슈

closes #(issue_num)


✅ 변경 사항

이번 PR에서 변경된 내용을 간략히 정리해주세요.

  • 기능1: 기존 버전은 composable이 직접 데이터를 들고 있었다. 뷰모델이 데이터를 가지고 있고 composable(UI)는 화면이 돌리면 화면이 새로 태어나므로 데이터를 가지고 있지 않는다는 원칙에 어긋났다. 그래서 ProfileViewModel.kt(데이터 보관하는 viewmodel)와 ProfileUiState.kt(화면이 가질 수 있는 3가지 상태 정해둠)를 새로 만들고, ProfileScreen.kt에서는 viewModel()로 상태관찰만 하도록 수정했다. (ProfileScreen.kt의 LaunchedEffect 방식에서 ViewModel을 재사용해 StatFlow값을 쓰는 방식으로 수정)
  • 기능 2: ProfileUiState.Error로 UI를 표시해 에러가 나면 무시하지 않고 사용자에게 에러 메시지를 띄우게 만들었고 사용자가 네트워크가 끊겯 다시 시도할 수 있게 만들었다.

📷 영상 및 스크린샷

image image image image

🔗 알게 된 사항

앱을 출시하려면 번들로 만들어야 한다.

📝 질문 사항

워크북과 과제를 하며 궁금했던 점을 작성해주세요

@hajinmn-stack hajinmn-stack requested a review from a team June 21, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant