Skip to content

[Fix] #32 JPA Auditing OffsetDateTime DateTimeProvider 미적용 버그 수정#34

Merged
yukyoungs merged 2 commits into
developfrom
fix/#32
Jul 22, 2026
Merged

[Fix] #32 JPA Auditing OffsetDateTime DateTimeProvider 미적용 버그 수정#34
yukyoungs merged 2 commits into
developfrom
fix/#32

Conversation

@yukyoungs

@yukyoungs yukyoungs commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

#️⃣ 관련 이슈

closed #32

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

🧩 작업 내용

  • BaseEntitycreatedAt, updatedAtOffsetDateTime 타입인데 JpaConfig가 기본 DateTimeProvider(LocalDateTime 기준)를 사용하고 있어 auditing 값이 의도대로 채워지지 않던 문제 수정
  • JpaConfigauditingDateTimeProvider 빈을 추가해 OffsetDateTime.now()를 직접 제공하도록 변경
  • @EnableJpaAuditing(dateTimeProviderRef = "auditingDateTimeProvider")로 명시적 연결

📸 스크린샷(선택)

📣 To Reviewers

Summary by CodeRabbit

  • 개선 사항
    • 생성일과 수정일이 현재 시각의 시간대 정보를 포함한 OffsetDateTime 형식으로 일관되게 기록되도록 개선했습니다.

@yukyoungs yukyoungs self-assigned this Jul 21, 2026
@yukyoungs yukyoungs added the 🛠️ Fix 버그 수정 label Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2fe00e59-ff80-4e78-aebb-3561a3d82433

📥 Commits

Reviewing files that changed from the base of the PR and between 6f586f0 and 58a2be8.

📒 Files selected for processing (1)
  • src/main/java/com/leets7th/job_is_be/global/config/JpaConfig.java

📝 Walkthrough

Walkthrough

JPA Auditing이 기본 DateTimeProvider 대신 OffsetDateTime.now()를 반환하는 auditingDateTimeProvider 빈을 사용하도록 JpaConfig 설정을 변경합니다.

Changes

JPA Auditing 설정

Layer / File(s) Summary
OffsetDateTime auditing provider 연결
src/main/java/com/leets7th/job_is_be/global/config/JpaConfig.java
DateTimeProvider 관련 의존성을 추가하고, auditingDateTimeProvider 빈을 등록한 뒤 @EnableJpaAuditing이 해당 빈을 참조하도록 변경합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: sky-0131

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 JPA Auditing에서 OffsetDateTime용 DateTimeProvider 적용 버그 수정이라는 핵심 변경을 정확히 요약합니다.
Linked Issues check ✅ Passed 직접 연결된 #32의 요구사항인 auditingDateTimeProvider 추가와 @EnableJpaAuditing 연결이 반영되어 있습니다.
Out of Scope Changes check ✅ Passed 변경은 JpaConfig 한 파일의 JPA Auditing 수정에 국한되어 있어 범위를 벗어난 변경이 보이지 않습니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@yeonjuncho yeonjuncho left a comment

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.

승인합니다

@yukyoungs
yukyoungs merged commit 5f63267 into develop Jul 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ Fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] JPA Auditing OffsetDateTime 미적용 버그 수정

2 participants