Reopen [Fix] 회원가입 플로우 오류 수정 #6

[ProtossManse]

🔗 관련 이슈

• Close [Fix] 회원가입시 어르신 건강정보 등록되지 않는 문제 #6

📙 작업 설명

• 로깅 라이브러리 Napier를 추가했습니다.
  • ktor client가 Napier를 사용하여 로깅하도록 변경했습니다.
• 회원가입 플로우 관련 여러가지 문제들을 해결했습니다.
  • 어르신이 등록되지 않고, 회원가입에만 성공했을 때 앱이 비정상적으로 종료되던 문제를 해결했습니다. ( 어르신이 없을 때, 빈 어르신 리스트를 인덱싱해서 Exception이 발생 )
  • 어르신 등록 시 birthDate 포맷이 잘못되어 400에러가 반환되던 문제를 해결했습니다.
  • 케어콜 시간 설정 화면의 uiState.isLoading 값이 업데이트 되지않아 화면이 Indicator만 표시하던 문제를 해결했습니다.
  • 케어콜 시간 설정 화면의 시간 설정 버튼이 uiState의 showBottomSheet 변수가 아닌, Screen 내부의 showBottonSheet 변수를 참조해 버튼을 눌러도 반응이 없던 문제를 해결하였습니다. (원래 uiState가 없었던 화면이라 uiState를 전체적으로 추가하면서 생긴 문제로 추정)

📸 스크린샷 또는 시연 영상 (선택)

기능	미리보기	기능	미리보기
기능 설명		기능 설명

💬 추가 설명 or 리뷰 포인트 (선택)

Summary by CodeRabbit

릴리스 노트

• Bug Fixes

  • 로딩 상태 처리 안정성 개선
  • 노인 정보 목록 조회 시 안정성 강화

• Chores

  • 애플리케이션 로깅 시스템 개선
  • 날짜 데이터 포맷팅 수정
  • UI 상태 관리 구조 최적화
  • 라이브러리 의존성 추가

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 79b361d8-4412-4f45-a042-a11acf284a48

📥 Commits

Reviewing files that changed from the base of the PR and between 0c71129 and c3c5222.

📒 Files selected for processing (9)

• composeApp/build.gradle.kts
• composeApp/src/androidMain/kotlin/com/konkuk/medicarecall/App.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/di/NetworkModule.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/mapper/ElderInfoMapper.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/mapper/ElderMapper.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/calltime/screen/CallTimeScreen.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/calltime/viewmodel/CallTimeViewModel.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/elder/screen/LoginElderInfoScreen.kt
• composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/elder/viewmodel/LoginElderViewModel.kt

---

Walkthrough

Napier 로깅 라이브러리 통합, Android 앱 초기화 시 로깅 활성화, Ktor HTTP 로깅을 Napier 어댑터로 변경, birthDate 형식화 수정, UI 상태 관리 리팩토링(로컬 상태를 uiState로 통합), 그리고 elder 데이터 유효성 검증 추가.

Changes

Cohort / File(s)	Summary
Napier 로깅 통합 composeApp/build.gradle.kts, composeApp/src/androidMain/kotlin/com/konkuk/medicarecall/App.kt, composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/di/NetworkModule.kt	Napier 로깅 라이브러리 종속성 추가, Android 앱 시작 시 DebugAntilog로 초기화, Ktor 기본 로거를 Napier로 라우팅하는 커스텀 Logger로 변경.
날짜 형식화 수정 composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/mapper/ElderInfoMapper.kt, composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/data/mapper/ElderMapper.kt	ElderInfo/Elder 매핑 시 birthDate를 formatAsDate()를 사용하여 문자열로 형식화.
UI 상태 관리 리팩토링 composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/calltime/screen/CallTimeScreen.kt, composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/calltime/viewmodel/CallTimeViewModel.kt	로컬 상태 변수를 uiState로 통합, showBottomSheet와 selectedTabIndex를 상태 관리에 포함, 로딩 상태 일관성 추가.
Elder 목록 유효성 검증 및 Import 정리 composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/elder/viewmodel/LoginElderViewModel.kt, composeApp/src/commonMain/kotlin/com/konkuk/medicarecall/ui/feature/login/elder/screen/LoginElderInfoScreen.kt	Elder 목록 비어있음 확인 후 업데이트, 명시적 import로 전환 및 import 정리.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• [Fix] 회원가입 플로우 오류 수정 #6 #7: Napier 로깅 통합, birthDate formatAsDate() 변환, 그리고 UI 상태/ViewModel 로직 조정을 포함하는 동일한 코드 레벨 변경사항들.

Suggested reviewers

• librawish808

Poem

🔍 로깅을 깔끔하게, Napier가 와서
📅 날짜도 포맷하고, 상태도 정리하고
✨ 어르신 정보가 이제 무사히 등록돼요
버그 하나씩 잡아가며, 코드는 더 튼튼해지고
🚀 회원가입 여정이 이제 완성됩니다!

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/elder-register-#6

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}