Skip to content

[REFACTOR] 아카이브 보드 이미지 페이징 조회 날짜 표기 수정#134

Merged
Chhun-Lee merged 2 commits intodevfrom
feat/#75_OAuth2_login
Feb 11, 2026
Merged

[REFACTOR] 아카이브 보드 이미지 페이징 조회 날짜 표기 수정#134
Chhun-Lee merged 2 commits intodevfrom
feat/#75_OAuth2_login

Conversation

@Chhun-Lee
Copy link
Collaborator

@Chhun-Lee Chhun-Lee commented Feb 11, 2026

🎋 이슈 및 작업중인 브랜치

🔑 주요 내용

  • 아카이브 보드 이미지 페이징 조회 날짜 표기 수정

Check List

  • Reviewers 등록을 하였나요?
  • Assignees 등록을 하였나요?
  • 라벨(Label) 등록을 하였나요?
  • PR 머지하기 전 반드시 CI가 정상적으로 작동하는지 확인해주세요!

Summary by CodeRabbit

변경 사항

  • 스타일
    • 아카이브 이미지의 업로드 시간 표시 방식이 상대적 시간(예: "2시간 전")에서 고정된 날짜 형식(YY.MM.DD)으로 변경되었습니다. 기존의 세부 시간 표기는 간소화되어 모든 업로드 항목에 일관된 날짜 형식으로 표시됩니다.

@Chhun-Lee Chhun-Lee self-assigned this Feb 11, 2026
@Chhun-Lee Chhun-Lee added the ♻️ REFACTOR 리팩토링 관련 라벨 label Feb 11, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

업로드 시간 포맷팅이 상대적 시간 계산(분/시간/일/주)에서 고정된 날짜 형식("yy.MM.dd")으로 변경되었고, DateTimeFormatter 상수(DATE_FORMATTER)가 도입되었습니다. 레코드 시그니처와 from(...) 호출은 유지되었습니다.

Changes

Cohort / File(s) Summary
응답 DTO 날짜 포맷팅
src/main/java/com/umc/nuvibe/domain/archive/dto/response/BoardImageResponse.java
formatUploadTime의 상대 시간 분기 로직을 제거하고 DateTimeFormatter 기반의 고정된 "yy.MM.dd" 포맷(DATE_FORMATTER)으로 변경. DateTimeFormatter 도입 및 관련 코드 조정. Lines changed: +5/-5.

Suggested reviewers

  • jaemin0413
🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 아카이브 보드 이미지의 날짜 표기 수정을 명확히 설명하며, 코드 변경 사항(DATE_FORMATTER 추가, 날짜 형식 변경)과 직접 관련이 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/#75_OAuth2_login

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In
`@src/main/java/com/umc/nuvibe/domain/archive/dto/response/BoardImageResponse.java`:
- Around line 32-33: Extract the DateTimeFormatter.ofPattern("yy.MM.dd") into a
static final constant in the BoardImageResponse class (e.g., private static
final DateTimeFormatter UPLOADED_AT_FORMATTER) and replace the inline call
uploadedAt.format(DateTimeFormatter.ofPattern("yy.MM.dd")) with
uploadedAt.format(UPLOADED_AT_FORMATTER); this avoids creating a new formatter
on each call and leverages the formatter's thread-safe, immutable nature.

@Chhun-Lee Chhun-Lee merged commit ba38ef7 into dev Feb 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ REFACTOR 리팩토링 관련 라벨

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant