[PR] 미션 결과 표시 형식 변경 및 버그 수정 (HomeTab#57)#62
Merged
Conversation
🐞fix: 잘못된 단위로 표시되던 미션 결과 데이터 수정
Closed
Contributor
There was a problem hiding this comment.
Code Review
이번 Pull Request에서는 Home 관련 ViewController들의 에러 핸들링 방식을 개선하여, 기존의 print 출력 대신 사용자에게 알럿(Alert)을 표시하도록 변경하였습니다. 또한, ViewModel에서 UI 표현을 위한 MissionResultList 구조체를 도입하여 날짜 포맷팅 로직을 셀(Cell)에서 뷰모델로 이동시켰으며, Rx closure 내에서 [weak self]를 사용하여 메모리 누수 가능성을 방지했습니다. 현재 리뷰 의견이 없으므로 별도의 피드백은 제공하지 않습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[PR] 미션 결과 표시 형식 변경 및 버그 수정 (HomeTab#57)
작업 내용
미션 결과의 데이터 가공 로직을 개선하고, 안정적인 에러 처리 및 프로그레스 계산 버그를 수정했습니다.
1. HomeViewModel 표시 형식 변경
2. 에러 처리 강화
HomeTab내 각 뷰 컨트롤러에서 데이터 로드 실패 시AlertController를 띄워 사용자에게 에러 상황을 명확히 알리도록 개선3. 버그 수정
detail)를 사용하도록 변경하여 정확한 수치가 반영되도록 수정관련 이슈