Conversation
Co-Authored-By: 김준서 Junseo Kim <semiwest2@gmail.com>
Co-Authored-By: 김준서 Junseo Kim <semiwest2@gmail.com>
Co-Authored-By: 김준서 Junseo Kim <semiwest2@gmail.com>
…plit-MOA-573 [feature] 앱 버전 관리를 위해 웹뷰 라우터를 분리한다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 코호트 / 파일(s) | 변경 요약 |
|---|---|
라우팅 및 앱 구성 frontend/src/App.tsx |
라우트 추가/수정: /club/:clubId → LegacyClubDetailPage, /clubDetail/:clubId → ClubDetailPage, /webview/club/:clubId → ClubDetailPage(웹뷰 전용). 레거시 처리 주석 추가 |
새 레거시 페이지 컴포넌트 frontend/src/pages/ClubDetailPage/LegacyClubDetailPage.tsx |
새로운 레거시 클럽 상세 페이지 추가: 헤더/프로필/탭(소개/피드)/푸터 구성, Mixpanel 추적, URL 쿼리 기반 탭 상태, 로딩/에러 처리 |
네비게이션 대상 업데이트 frontend/src/pages/ApplicationFormPage/ApplicationFormPage.tsx, frontend/src/pages/MainPage/components/ClubCard/ClubCard.tsx |
네비게이션 경로를 /club/{clubId}에서 /clubDetail/{clubId}로 변경(오류 리다이렉션·성공 후 교체 네비게이션 등) |
UI 동작 조건 변경 frontend/src/pages/ClubDetailPage/components/ClubApplyButton/ClubApplyButton.tsx |
디바이스 감지 추가(useDevice)로 공유 버튼 노출 제어(모바일/태블릿에서는 숨김) |
Sequence Diagram(s)
mermaid
sequenceDiagram
participant User as 사용자(브라우저)
participant App as App.tsx (라우터)
participant Legacy as LegacyClubDetailPage
participant NewPage as ClubDetailPage
participant NavSrc as Nav Components (ClubCard / ApplicationForm)
User->>App: 요청 /club/:clubId 또는 /clubDetail/:clubId 또는 /webview/club/:clubId
App-->>Legacy: 경로 /club/:clubId 매칭 → LegacyClubDetailPage 렌더
App-->>NewPage: 경로 /clubDetail/:clubId 또는 /webview/club/:clubId 매칭 → ClubDetailPage 렌더
NavSrc->>App: 네비게이트 /clubDetail/:id (클릭/폼 제출 후 리다이렉트)
App-->>NewPage: 해당 경로로 렌더
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related issues
- [feature] MOA-573 앱 버전 관리를 위해 웹뷰 라우터를 분리한다 #1113:
/webview/club/:clubId라우팅 추가로 웹뷰 라우터 분리 목적과 직접 연결됨
Possibly related PRs
- [feature] 앱 버전 관리를 위해 웹뷰 라우터를 분리한다 #1114: 동일 파일들(라우팅·LegacyClubDetailPage·네비게이션 변경)을 수정한 PR로 직접적인 코드 중복/충돌 가능
- [release] v1.0.9 #607: ClubApplyButton 및 클럽 상세 페이지 동작을 함께 수정하는 PR로 연관성 높음
- [fix] 지원서 미등록 시 alert 반복 버그 수정 및 외부 링크/지원서 분기 처리 #605: ClubApplyButton 변경(구성/동작)과 관련된 PR로 코드 레벨 연관성 있음
Suggested reviewers
- seongwon030
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Title check | ❓ Inconclusive | PR 제목이 변경사항의 주요 내용을 반영하지 못하고 있습니다. '[release] FE v1.1.20'은 버전 정보만 포함하며, 실제 구현된 주요 변경사항(레거시 페이지 분리, 라우팅 경로 변경, 웹뷰 지원)을 구체적으로 설명하지 않습니다. | PR 제목을 더 구체적으로 수정하여 주요 변경사항을 명확히 설명하시기 바랍니다. 예를 들어 '[release] FE v1.1.20: 레거시 클럽 상세 페이지 추가 및 라우팅 경로 개선'과 같이 변경사항의 핵심을 포함하는 것이 좋습니다. |
✅ Passed checks (2 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
이유: 모바일 및 태블릿 상세페이지는 전용 탑바(ClubDetailTopBar)를 통해 공유 기능을 제공하도록 UX가 개편됨에 따라, 하단 스티키 푸터(ClubDetailFooter)에서의 공유 버튼 중복 노출을 방지하고 레이아웃을 간소화하기 위함.
#️⃣연관된 이슈
📝작업 내용
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
새로운 기능
버그 수정 / 개선
✏️ Tip: You can customize this high-level summary in your review settings.