Dailyrecord#102
Closed
hyeonjiana wants to merge 12 commits into
Closed
Conversation
- **API 예외 처리 강화**: `MindRecordRepositoryImpl`의 `delete`, `get`, `edit`, `setMindRecordReceiverEnabled` 메서드에서 API 응답이 성공적이지 않을 경우 `HttpException`을 던지도록 수정하여 예외 처리를 강화했습니다.
- **수신자 전달 설정 기능 추가**:
- 특정 마음의 기록을 특정 수신자에게 전달하거나 중단하는 `setMindRecordReceiverEnabled` 기능을 `MindRecordRepository`와 `DailyRecordApiService`에 추가했습니다 (`PATCH /mind-records/{recordId}/receivers/{receiverId}`).
- 해당 기능을 비즈니스 로직으로 감싸는 `SetMindRecordReceiverEnabledUseCase`를 새로 추가했습니다.
- **ViewModel 파라미터 리팩토링**: `MindRecordViewModel`의 `onCreateRecord`와 `editRecord` 메서드 파라미터를 각각 `CreateRecordParams` 및 `EditRecordParams` 데이터 클래스로 묶어 코드 가독성과 유지보수성을 개선했습니다.
- **저장 실패 에러 상태 추가**: `MindRecordWriterViewModel`이 저장에 실패했을 때 에러 메시지를 UI에 표시할 수 있도록 `MindRecordUiState`에 `saveErrorMessage` 필드를 추가하고 관련 로직을 업데이트했습니다.
- **로그 및 주석 개선**: ViewModel 전반에 걸쳐 일관된 `TAG`를 사용하고, 예외 발생 시 더 상세한 정보를 포함하도록 로그를 개선했습니다.
…nto dailyrecord
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
2 participants
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.
#️⃣연관된 이슈
📝작업 내용
스크린샷 (선택)
💬리뷰 요구사항(선택)