Skip to content

refactor: 인기차 섹션 검색-탐색 이동, 더보기 메뉴 확장, 캘린더 헤더 통일#266

Merged
FLYLIKEB merged 1 commit into
mainfrom
feature/ui-restructure-trending-calendar-more-menu
Mar 16, 2026
Merged

refactor: 인기차 섹션 검색-탐색 이동, 더보기 메뉴 확장, 캘린더 헤더 통일#266
FLYLIKEB merged 1 commit into
mainfrom
feature/ui-restructure-trending-calendar-more-menu

Conversation

@FLYLIKEB
Copy link
Copy Markdown
Owner

@FLYLIKEB FLYLIKEB commented Mar 16, 2026

Summary

  • 홈 인기차/인기다우 섹션을 검색-탐색(/sasaek) 탐색 탭으로 이동 (컴팩트 UI)
  • 더보기 메뉴에 탐색, 알림(인증시), 캘린더 항목 추가
  • 캘린더 헤더에 showBack 추가하여 다른 페이지와 UI 통일
  • HomeTrendingSection 삭제, Home fetchForYouFeed 정리

Test plan

  • npm run build 통과
  • npm run test:run — 관련 테스트 26건 통과 (Home, Search, TeaCalendar, MoreMenu, BottomNav)
  • 홈 페이지에서 인기차 섹션 미표시 확인
  • /sasaek 탐색 탭에서 인기차/인기다우 섹션 표시 확인
  • 더보기 메뉴에서 캘린더, 알림, 탐색 접근 가능 확인
  • 캘린더 페이지 뒤로가기 버튼 동작 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 더보기 메뉴에 탐색, 알림, 차록 캘린더 항목 추가
    • 탐색 탭에 인기 차 및 인기 크리에이터 섹션 추가
    • 차록 캘린더 페이지에 뒤로가기 네비게이션 추가
  • 테스트

    • 더보기 메뉴 및 탐색 페이지 테스트 추가
    • 차록 캘린더 뒤로가기 버튼 테스트 추가

- 홈 인기차/인기다우 섹션을 검색-탐색(/sasaek) 탐색 탭으로 이동
- 인기차 카드 크기 컴팩트하게 변경 (300px → 200px)
- 더보기 메뉴에 탐색, 알림(인증시), 캘린더 항목 추가
- 캘린더 헤더에 showBack 추가하여 다른 페이지와 통일
- HomeTrendingSection 삭제 (orphaned)
- Home fetchForYouFeed에서 불필요한 Promise.allSettled 제거

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cha-log Ready Ready Preview, Comment Mar 16, 2026 3:43am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 16, 2026

Caution

Review failed

Pull request was closed or merged during review

워크스루

홈 페이지에서 추세 섹션을 제거하고, 탐색 페이지로 이동시켰습니다. 더보기 메뉴에 탐색과 캘린더 경로를 추가하고, 인증된 사용자에게만 알림 메뉴를 표시하도록 했습니다. 메뉴 활성화 로직을 단순화하고 관련 라우팅과 테스트를 업데이트했습니다.

변경 사항

코호트 / 파일 요약
메뉴 및 라우팅
src/components/BottomNav.tsx, src/components/MoreMenu.tsx, src/components/__tests__/MoreMenu.test.tsx
MoreMenu에 탐색(/sasaek)과 캘린더(/calendar) 항목 추가, 인증 여부에 따른 조건부 알림 메뉴 표시. BottomNav의 MORE_PATHS에 새 경로 추가. 메뉴 활성화 로직 단순화 및 단위 테스트 추가.
컴포넌트 제거 및 재배치
src/components/HomeTrendingSection.tsx, src/components/search/ExploreSection.tsx
HomeTrendingSection 컴포넌트 삭제(53줄). ExploreSection에 trendingTeas와 trendingCreators 옵션 props 추가하여 추세 콘텐츠 조건부 렌더링.
페이지 업데이트
src/pages/Home.tsx, src/pages/Search.tsx, src/pages/TeaCalendar.tsx
Home에서 추세 관련 상태 및 API 호출 제거. Search의 탐색 탭에서 추세 데이터 조회 추가. TeaCalendar에 뒤로 가기 버튼 표시 추가.
테스트 업데이트
src/pages/__tests__/Home.test.tsx, src/pages/__tests__/Search.test.tsx, src/pages/__tests__/TeaCalendar.test.tsx
Home 테스트에서 인기 차 섹션 비표시 확인. Search 테스트에서 usersApi 모킹 및 인기 차 섹션 표시 확인 추가(중복된 테스트 발견). TeaCalendar 테스트에 뒤로 가기 버튼 렌더링 확인 추가.

시퀀스 다이어그램

sequenceDiagram
    participant User
    participant SearchPage as Search 페이지
    participant API as API
    participant ExploreSection as ExploreSection<br/>컴포넌트

    User->>SearchPage: 탐색 탭 클릭
    activate SearchPage
    SearchPage->>API: teasApi.getTrending('7d') 요청
    API-->>SearchPage: 인기 차 데이터 반환
    SearchPage->>API: usersApi.getTrending('7d') 요청
    API-->>SearchPage: 인기 creator 데이터 반환
    SearchPage->>SearchPage: state에 데이터 저장
    SearchPage->>ExploreSection: trendingTeas, trendingCreators props 전달
    activate ExploreSection
    ExploreSection->>ExploreSection: 인기 차와 creator 섹션 렌더링
    ExploreSection-->>User: 추세 콘텐츠 표시
    deactivate ExploreSection
    deactivate SearchPage
Loading

예상 코드 리뷰 노력

🎯 3 (Moderate) | ⏱️ ~25 분

관련 가능성이 있는 PR

🚥 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 제목이 변경 내용의 주요 사항을 명확하게 요약하고 있습니다: 인기차 섹션 이동, 더보기 메뉴 확장, 캘린더 헤더 통일이 정확히 반영되어 있습니다.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/ui-restructure-trending-calendar-more-menu
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@FLYLIKEB FLYLIKEB merged commit b099ad2 into main Mar 16, 2026
2 of 3 checks passed
@FLYLIKEB FLYLIKEB deleted the feature/ui-restructure-trending-calendar-more-menu branch March 16, 2026 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant