Skip to content

fix: Diary 날짜 필드의 Timezone을 정상적으로 처리하도록 수정 - #649

Merged
qdrptd merged 2 commits into
developfrom
fix/diary-date-timezone-parsing
Sep 12, 2026
Merged

qdrptd merged 2 commits into
developfrom
fix/diary-date-timezone-parsing

Conversation

@JuTaK97

@JuTaK97 JuTaK97 commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

개요

서버 스펙 변경(PR #548)으로 인해 DiarySubmissionSummaryDto의 date 필드가 timezone을 포함한 문자열로 변경됨에 따라,
기존 LocalDateTime.parse(date)에서 DateTimeParseException이 발생할 수 있는 문제를 방지합니다.

작업 내용

  • LocalDateTime.parse(date) 대신 내부적으로 Instant를 사용하는 getLocalDateTimeFromString(date)를 사용하도록 수정

기타

  • 이 PR이 앱의 다음 릴리스에 포함되어야 클라이언트 크래시를 방지할 수 있습니다.

서버 스펙 변경(PR #548)으로 인해 DiarySubmissionSummaryDto의 date 필드가 timezone을 포함한 문자열로 변경됨에 따라,
기존 LocalDateTime.parse(date)에서 DateTimeParseException이 발생할 수 있는 문제를
getLocalDateTimeFromString(date) (내부적으로 Instant 사용)을 사용하도록 수정하여 방지함.
@JuTaK97
JuTaK97 requested a review from a team as a code owner September 6, 2026 11:36
Comment on lines +85 to +87
Handler(Looper.getMainLooper()).postDelayed({
throw IllegalStateException("Crashlytics test crash: invalid startup state")
}, 3_000)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

테스트가 들어있어용

@qdrptd
qdrptd merged commit 73575a5 into develop Sep 12, 2026
2 checks passed
@qdrptd
qdrptd deleted the fix/diary-date-timezone-parsing branch September 12, 2026 05:49
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.

3 participants