Skip to content

feat: 피드 날짜 수정#852

Open
Sadturtleman wants to merge 2 commits intodevelopfrom
feat/842
Open

feat: 피드 날짜 수정#852
Sadturtleman wants to merge 2 commits intodevelopfrom
feat/842

Conversation

@Sadturtleman
Copy link
Copy Markdown
Contributor

@Sadturtleman Sadturtleman commented Mar 29, 2026

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

image image

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

피드 날짜 정책 수정으로 feeduimodel의 formmateddate 제거하고 createddate그대로 가져오는 걸로 변경했습니다

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 피드 항목의 날짜 처리 방식을 단순화하여 날짜 표시가 더 일관되고 안정적으로 동작하도록 수정했습니다. 사용자에게 보이는 날짜 표현에는 변화가 없으며 내부 구현이 정리되었습니다.

@Sadturtleman Sadturtleman added 🍯 [FEAT] 새로운 기능을 개발합니다. 🧙마법사 하루 labels Mar 29, 2026
@github-actions github-actions bot requested review from devfeijoa, m6z1 and s9hn March 29, 2026 10:37
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 470664bb-1771-4876-a1b2-395d49e322e9

📥 Commits

Reviewing files that changed from the base of the PR and between 39a8255 and 62b8449.

📒 Files selected for processing (1)
  • feature/feed/src/main/java/com/into/websoso/feature/feed/component/FeedSection.kt
✅ Files skipped from review due to trivial changes (1)
  • feature/feed/src/main/java/com/into/websoso/feature/feed/component/FeedSection.kt

Walkthrough

FeedUiModel의 formattedCreatedDate 계산 속성을 삭제하고, FeedSection에서 피드 생성일 표시에 feed.createdDate를 직접 사용하도록 변경했습니다. 관련 미사용 임포트도 제거되었습니다.

Changes

Cohort / File(s) Summary
피드 모델 변경
feature/feed/src/main/java/com/into/websoso/feature/feed/model/FeedUiModel.kt
val formattedCreatedDate: String 게터 기반 계산 속성 제거 및 관련 미사용 import 삭제.
피드 UI 변경
feature/feed/src/main/java/com/into/websoso/feature/feed/component/FeedSection.kt
FeedItem의 생성일 텍스트를 feed.formattedCreatedDatefeed.createdDate로 교체 (표시 소스 변경, 로직/플로우는 없음).

Sequence Diagram(s)

(생략)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분

Suggested reviewers

  • s9hn
  • devfeijoa

Poem

당근 입에 문 토끼가 말하네 🥕
날짜를 깔끔히 떼어냈더니
보기도 쉽고 마음도 편하네
짧은 문자열, 큰 휴식
피드는 다시 폴짝 폴짝 뛰네 🐇

🚥 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
Title check ✅ Passed PR 제목 '피드 날짜 수정'은 변경 사항의 주요 내용을 명확하게 반영하며, 피드의 날짜 표시 정책 수정이라는 핵심 변경을 간결하게 표현합니다.
Description check ✅ Passed PR 설명은 템플릿 구조를 따르고 연결된 이슈(#842)를 명시했으며, 스크린샷과 검토자 메모를 포함했으나 '작업 설명(Work Description)' 섹션은 미작성 상태입니다.
Linked Issues check ✅ Passed 코드 변경사항(FeedUiModel에서 formattedCreatedDate 제거, FeedSection에서 createdDate 직접 사용)이 연결된 이슈 #842의 요구사항 '피드 날짜 정책 수정'을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 코드 변경사항은 피드 날짜 표시 정책 수정이라는 PR 목표 범위 내에 있으며, 무관한 변경사항이 없습니다.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/842

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
Copy Markdown
Member

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

좋아요👍

Copy link
Copy Markdown
Contributor

@devfeijoa devfeijoa left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍 린트 때문에 CI 실패한것 같은데 그것만 확인 부탁드려요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍯 [FEAT] 새로운 기능을 개발합니다. 🧙마법사 하루

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 피드 날짜 수정

3 participants