Skip to content

[PR] 미션 결과 표시 형식 변경 및 버그 수정 (HomeTab#57)#62

Merged
devBambu merged 5 commits into
developfrom
feature/HomeTab#57
Mar 30, 2026
Merged

[PR] 미션 결과 표시 형식 변경 및 버그 수정 (HomeTab#57)#62
devBambu merged 5 commits into
developfrom
feature/HomeTab#57

Conversation

@devBambu
Copy link
Copy Markdown
Collaborator

[PR] 미션 결과 표시 형식 변경 및 버그 수정 (HomeTab#57)

작업 내용

미션 결과의 데이터 가공 로직을 개선하고, 안정적인 에러 처리 및 프로그레스 계산 버그를 수정했습니다.

1. HomeViewModel 표시 형식 변경

  • MissionResultList: UI 표현에 최적화된 데이터 모델 정의 (날짜 포맷팅, 성공 여부 등)
  • DateFormatter: "M월 dd일 (E)" 형식의 로컬라이즈된 날짜 변환 로직 적용하여 사용자 친화적인 정보 제공

2. 에러 처리 강화

  • AlertController 적용: HomeTab 내 각 뷰 컨트롤러에서 데이터 로드 실패 시 AlertController를 띄워 사용자에게 에러 상황을 명확히 알리도록 개선

3. 버그 수정

  • ProgressView 계산 로직 수정: 프로그레스 달성률 계산 시 입력값을 시간(Hour)이 아닌 분(Minute) 단위 데이터(detail)를 사용하도록 변경하여 정확한 수치가 반영되도록 수정

관련 이슈

@devBambu devBambu linked an issue Mar 30, 2026 that may be closed by this pull request
@devBambu devBambu self-assigned this Mar 30, 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

이번 Pull Request에서는 Home 관련 ViewController들의 에러 핸들링 방식을 개선하여, 기존의 print 출력 대신 사용자에게 알럿(Alert)을 표시하도록 변경하였습니다. 또한, ViewModel에서 UI 표현을 위한 MissionResultList 구조체를 도입하여 날짜 포맷팅 로직을 셀(Cell)에서 뷰모델로 이동시켰으며, Rx closure 내에서 [weak self]를 사용하여 메모리 누수 가능성을 방지했습니다. 현재 리뷰 의견이 없으므로 별도의 피드백은 제공하지 않습니다.

@devBambu devBambu merged commit 05df132 into develop Mar 30, 2026
1 of 2 checks passed
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.

[FEAT]: 에러 처리 및 버그 수정

1 participant