Skip to content

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

Merged
qdrptd merged 4 commits into
developfrom
restore-diary-timezone-without-crash
Sep 13, 2026
Merged

fix: Diary 날짜 필드의 Timezone을 정상적으로 처리하도록 수정#651
qdrptd merged 4 commits into
developfrom
restore-diary-timezone-without-crash

Conversation

@qdrptd

@qdrptd qdrptd commented Sep 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

PR #650이 #649를 revert한 뒤 diary timezone 수정만 다시 적용합니다. 원 PR에 섞였던 startup Crashlytics test 커밋은 되돌려 제외했습니다.

JuTaK97 and others added 3 commits September 6, 2026 20:35
@JuTaK97
fix: DiarySubmissionSummaryDto 날짜 필드의 Timezone을 정상적으로 처리하도록 수정
ee849d5 
서버 스펙 변경(PR #548)으로 인해 DiarySubmissionSummaryDto의 date 필드가 timezone을 포함한 문자열로 변경됨에 따라,
기존 LocalDateTime.parse(date)에서 DateTimeParseException이 발생할 수 있는 문제를
getLocalDateTimeFromString(date) (내부적으로 Instant 사용)을 사용하도록 수정하여 방지함.
@qdrptd
Update startup crash test configuration
cb19618
@qdrptd
Revert "Update startup crash test configuration"
8d34953 
This reverts commit cb19618.
Copilot AI lite review requested due to automatic review settings September 13, 2026 04:00
@qdrptd
qdrptd requested a review from a team as a code owner September 13, 2026 04:00
Copilot AI reviewed Sep 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@qdrptd
qdrptd enabled auto-merge (squash) September 13, 2026 04:00
@qdrptd
qdrptd disabled auto-merge September 13, 2026 04:04
@qdrptd
qdrptd merged commit e3ccc70 into develop Sep 13, 2026
2 checks passed
@qdrptd
qdrptd deleted the restore-diary-timezone-without-crash branch September 13, 2026 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@qdrptd @JuTaK97