[Feature/#285] 타임라인 상세조회 API 응답에 comparisonPeriodType 반영 및 수정 prefilling

[jjjsun]

🚨 관련 이슈

Closed #285

✨ 변경사항

• 🐞 BugFix Something isn't working
• 💻 CrossBrowsing Browser compatibility
• 🌏 Deploy Deploy
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

• TimelineDetail에 comparisonPeriodType 피드 추가
• 타임라인ㅇ 수정 모달 prefilling 시 상세 API 응답값 사용 (LAST_WEEK 하드코딩 제거)
• TIMELINE_DETAIL_MOCK에 comparisonPeriodType 반영

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

생성 모달에서의 기본값은 LAST_WEEK로 변경없습니다.

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

• Bug Fixes
  • 타임라인 수정 화면에서 비교 기간 타입이 상세 데이터와 일치하도록 반영되도록 개선했습니다.
  • 타임라인 상세 정보에 비교 기간 타입이 포함되어, 편집 시 더 정확한 값이 표시됩니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 88158917-b241-4204-96fa-9e917483a47d

📥 Commits

Reviewing files that changed from the base of the PR and between 574e52c and 991e70a.

📒 Files selected for processing (3)

• src/pages/dashboard/timeline/Timeline.tsx
• src/types/timeline/api.ts
• src/types/timeline/timeline.mock.ts

---

📝 Walkthrough

Walkthrough

타임라인 상세 응답 타입(ITimelineDetail)과 mock 데이터에 comparisonPeriodType 필드를 추가하고, Timeline 컴포넌트의 수정 모달 초기값 구성에서 하드코딩된 "LAST_WEEK" 대신 상세 데이터의 comparisonPeriodType 값을 사용하도록 변경했습니다.

Changes

comparisonPeriodType 반영

Layer / File(s)	Summary
타입 및 mock 데이터 확장 src/types/timeline/api.ts, src/types/timeline/timeline.mock.ts	ITimelineDetail에 comparisonPeriodType 필드가 추가되고, TIMELINE_DETAIL_MOCK에도 동일 필드가 "LAST_WEEK"로 채워졌습니다.
수정 모달 초기값 로직 변경 src/pages/dashboard/timeline/Timeline.tsx	editInitialValues 생성 시 comparisonPeriodType을 하드코딩값 대신 editDetail.comparisonPeriodType으로 설정합니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related issues

• fix: 타임라인 상세조회 - comparisonPeriodType추가 WhereYouAd-Backend#168: 백엔드에서 타임라인 상세 API 응답에 comparisonPeriodType을 추가하는 이슈로, 본 PR의 FE 타입/모달 반영 작업과 직접 연결됩니다.

Possibly related PRs

• WhereYouAd/WhereYouAd-Frontend#251: 동일하게 ITimelineDetail의 comparisonPeriodType 필드 및 모달 초기값 로직을 다룹니다.
• WhereYouAd/WhereYouAd-Frontend#253: TTimelineComparisonPeriodType 및 ITimelineDetail 정의를 함께 다루는 동일 타입 영역 변경입니다.
• WhereYouAd/WhereYouAd-Frontend#283: TimelineCreateModal의 수정 모드 initialValues 처리와 본 PR의 초기값 로직이 함께 맞물립니다.

Suggested labels: 📬 API

Suggested reviewers: YermIm, Seojegyeong

수정 모달 초기값 로직을 보니, 하드코딩 제거는 좋은 방향이네요. 다만 editDetail.comparisonPeriodType이 undefined일 가능성이 있는지 타입 레벨과 런타임 모두 확인해 보시면 좋겠습니다. 상세 응답이 항상 이 필드를 포함한다는 보장이 BE 계약에 명시되어 있는지도 함께 체크해 주세요. 전체적으로 변경 범위가 작고 명확해서 리뷰 부담은 크지 않습니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	타임라인 상세 응답의 comparisonPeriodType 반영과 수정 모달 prefilling이라는 핵심 변경을 잘 요약하고 있습니다.
Description check	✅ Passed	이슈, 변경사항, 작업 내용, 미완성 작업, 참고 사항이 모두 포함되어 있어 템플릿 요구를 대부분 충족합니다.
Linked Issues check	✅ Passed	상세 API에 comparisonPeriodType을 추가하고 수정 모달 초기값에 응답값을 사용해 #285의 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	변경 범위가 timeline 상세 응답, 수정 모달 prefilling, mock 갱신으로 한정되어 있어 불필요한 추가 작업은 보이지 않습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ 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 feature/#285

---

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.}

[github-actions]

📚 Storybook 배포 완료

항목	링크
📖 Storybook	https://69a147b60a56365d9e2185ef-fixxiuttcc.chromatic.com/
🔍 Chromatic	https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=398

[YermIm]

P4: 확인했습니다!