#67 ✨ feat: 기사 상세 조회 시 날짜 포멧 통일 및 썸네일과 원문 링크 응답에 추가

[kingmingyu]

📍 PR 타입 (하나 이상 선택)

• 기능 추가
• 버그 수정
• 의존성, 환경 변수, 빌드 관련 코드 업데이트
• 기타 사소한 수정

❗️ 관련 이슈 링크

Close #67

📌 개요

• 기사 상세 조회 시 날짜 포멧 통일과 썸네일 url, 원문 링크를 응답에 추가하였습니다.

🔁 변경 사항

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

• PR 템플릿에 맞추어 작성했어요.
• 변경 내용에 대한 테스트를 진행했어요.
• 프로그램이 정상적으로 동작해요.
• PR에 적절한 라벨을 선택했어요.
• 불필요한 코드는 삭제했어요.

Summary by CodeRabbit

릴리스 노트

• 새로운 기능
  • 뉴스 상세 정보에 썸네일 이미지 URL과 원본 기사 URL 필드 추가
  • 뉴스 날짜 표시를 한국식 형식으로 개선

[coderabbitai]

Walkthrough

뉴스 상세 조회 응답 DTO에 썸네일 URL과 원문 URL 필드를 추가하고, 날짜를 한국식 형식으로 포맷하는 로직을 구현했습니다. 전체 2개 파일에서 20줄의 변경이 발생했습니다.

Changes

코호트 / 파일(들)	요약
응답 DTO 필드 확장 src/main/java/com/finsight/finsight/domain/learning/application/dto/response/LearningResponseDTO.java	LearningNewsDetailResponse 레코드에 thumbnailUrl과 originalUrl 필드 추가. 뉴스 상세 조회 응답에 원문 링크와 썸네일 이미지 URL 정보 포함.
날짜 포맷팅 로직 추가 src/main/java/com/finsight/finsight/domain/learning/persistence/mapper/LearningConverter.java	KOREAN_DATE_FORMATTER 날짜 포맷터 추가 및 formatKoreanDate() 헬퍼 메서드 구현. NewsDetailResponse.date 필드를 한국식 형식으로 포맷하도록 수정.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분

Suggested reviewers

• ramen0519
• kkshyun

---

👀 검토 시 주목할 점

칭찬할 부분:

• ✅ 날짜 포맷팅을 별도의 헬퍼 메서드로 분리해서 가독성과 재사용성을 고려한 좋은 구현입니다.
• ✅ DTO에 필드를 추가할 때 기존 필드들과의 일관성을 유지한 것이 좋습니다.

확인이 필요한 부분:

• ⚠️ null 체크: formatKoreanDate() 메서드에서 LocalDateTime 인자가 null인 경우의 처리가 있는지 확인해주세요. null이 전달되면 NullPointerException이 발생할 수 있습니다.
• ⚠️ 선택적 필드: thumbnailUrl과 originalUrl이 항상 존재하는지, 아니면 null이 가능한지 명확히 해야 합니다. 데이터 모델과 API 사용자 입장에서 예상치 못한 null 값이 없도록 확인해주세요.
• ⚠️ 날짜 포맷: KOREAN_DATE_FORMATTER 패턴이 실제 요청사항과 일치하는지 (예: "2026년 2월 15일" 형식인지 "2026-02-15" 형식인지) 링크된 이슈 #67과 대조해 확인해주세요.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 75.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
Title check	✅ Passed	PR 제목이 변경 사항을 명확하게 요약하고 있습니다. 날짜 포맷 통일과 썸네일/원문 링크 추가라는 주요 변경 사항을 정확히 반영하고 있습니다.
Description check	✅ Passed	PR 설명이 템플릿을 따르고 있으며, 개요, 관련 이슈, 테스트 체크리스트를 포함하고 있습니다. 다만 '변경 사항' 섹션이 상세하지 않습니다.
Linked Issues check	✅ Passed	PR이 이슈 #67의 모든 요구 사항을 충족합니다. DTO에 originalUrl과 thumbnailUrl을 추가했고, LearningConverter에서 한글 날짜 포맷을 적용하여 두 가지 목표를 모두 달성했습니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 이슈 #67의 범위 내에 있습니다. DTO 필드 추가와 날짜 포맷 통일이라는 명확한 목표에만 집중하고 있습니다.

_{✏️ 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/#67

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

• ✨ 136.113.208.247/32 (new)
• 34.170.211.100/32
• 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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