Skip to content

[#487] ProfileViewModel 생성 및 생명주기를 개선한다#497

Merged
opficdev merged 4 commits into
developfrom
fix/#487-ProfileViewModel
May 27, 2026
Merged

[#487] ProfileViewModel 생성 및 생명주기를 개선한다#497
opficdev merged 4 commits into
developfrom
fix/#487-ProfileViewModel

Conversation

@opficdev
Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • Profile 탭의 데이터 로드 시점을 다른 탭 화면과 동일하게 탭 선택 상태 기준으로 맞추기 위함
  • 화면 재렌더링 시점의 불필요한 fetch를 줄이고, 액션 이름의 의미를 실제 동작과 일치

📝 작업 내용

📌 요약

  • Profile 탭의 초기/재진입 데이터 로드 트리거를 onAppear에서 MainView의 탭 선택 변경 시점으로 이동
  • ProfileViewModel의 루트 진입 액션명을 onAppear에서 fetchData로 변경
  • 변경된 Home 액션명에 맞춰 테스트 코드 수정
  • Localizable.xcstrings 추출 상태 메타데이터 동기화
  • docs/DevLog.drawio 시퀀스 다이어그램 수정

🔍 상세

  • MainView에서 selectedTab 변경 시 .profile 선택에 대해 profileViewCoordinator.fetchData() 호출 추가
  • ProfileView.onAppear { coordinator.viewModel.send(...) } 제거
  • ProfileViewCoordinator.fetchData() 추가 및 내부 호출을 viewModel.send(.fetchData)로 정리
  • ProfileViewModel.ActiononAppearfetchData로 변경하고 reducer 분기명 정리
  • DeleteWebPageTests에서 변경된 HomeViewModel 액션명(.fetchData) 반영
  • Localizable.xcstrings의 키 재정렬 및 extractionState 메타데이터 갱신 반영

📸 영상 / 이미지 (Optional)

2026-05-27.9.47.57.mov
2026-05-27.9.53.22.mov
개선 전 개선 후

@opficdev opficdev self-assigned this May 27, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이번 풀 요청은 MainView에서 탭 전환 시 코디네이터를 통해 프로필 데이터를 가져오도록 .onAppear 이벤트를 .fetchData 액션으로 리팩토링하고, 관련 테스트 코드, 로컬라이징 문자열 및 DevLog.drawio 다이어그램을 업데이트합니다. 리뷰 피드백에서는 탭을 전환할 때마다 fetchData가 무조건 호출되어 불필요한 네트워크 요청이 발생하는 성능 문제를 지적하며, 데이터가 비어있을 때만 요청을 수행하도록 개선할 것을 제안했습니다.

@opficdev opficdev merged commit aa46061 into develop May 27, 2026
1 check passed
@opficdev opficdev deleted the fix/#487-ProfileViewModel branch May 27, 2026 01:15
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.

ProfileViewModel 생성 및 생명주기를 개선한다

1 participant