[release] FE v1.7.3#1552
Conversation
- festivalTimetableNavExperiment 실험 제거, tabs UI로 고정 - 대동제 필터 칩 및 /festival-busking, /festival-introduction 라우트 제거 - Filter.tsx의 daedongDotSeen 세션스토리지 로직 제거
- MainPage: DAEDONG_POPUP → APP_DOWNLOAD_POPUP 복구 - WebviewMainPage: 팝업 제거 (원래 없던 상태로 복원) - popupConfigs: DAEDONG_POPUP 및 daedong 이미지 import 제거
…daedong-MOA-858 [feature] 대동제 종료 — A/B 테스트 정리 및 관련 코드 제거
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. Warning
|
| Layer / File(s) | Summary |
|---|---|
Experiment definition cleanup frontend/src/experiments/definitions.ts |
ALL_EXPERIMENTS 배열을 비워 미사용 페스티벌 네비게이션 실험 정의를 제거합니다. |
BuskingPage navigation UI and tracking simplification frontend/src/pages/FestivalPage/BuskingPage/BuskingPage.tsx |
실험 기반 조건부 네비게이션을 제거하고 항상 DayTabsNav를 표시하도록 단일화하며, Mixpanel 추적 이벤트 페이로드에서 nav_variant 필드를 제거하고 day 변경 정보 중심으로 단순화합니다. |
Filter component session state and notification logic cleanup frontend/src/components/common/Filter/Filter.tsx |
useState 기반 세션 저장 상태 관리를 제거하고, WEB_FILTER_OPTIONS에서 "대동제" 옵션을 삭제하며, NotificationDot 표시 조건을 "홍보" 필터에만 한정하여 단순화합니다. |
Main page popup configuration update frontend/src/pages/MainPage/MainPage.tsx |
팝업 설정을 DAEDONG_POPUP에서 APP_DOWNLOAD_POPUP으로 변경하고 관련 import를 업데이트합니다. |
Sequence Diagram(s)
sequenceDiagram
participant User
participant BuskingPage
participant Mixpanel
User->>BuskingPage: Swipe or change day
BuskingPage->>BuskingPage: Calculate duration (DAEDONG2026_DAY_DURATION)
Note over BuskingPage: Remove nav_variant from payload
BuskingPage->>Mixpanel: Track event (duration only)
BuskingPage->>BuskingPage: Track day change (DAEDONG2026_DAY_CHANGED)
Note over BuskingPage: Remove nav_variant from payload
BuskingPage->>Mixpanel: Track event (from_day, to_day, interaction)
sequenceDiagram
participant MainPage
participant Popup
participant popupConfigs
MainPage->>popupConfigs: Import APP_DOWNLOAD_POPUP
Note over MainPage: Changed from DAEDONG_POPUP config
MainPage->>Popup: Pass APP_DOWNLOAD_POPUP config
Popup->>Popup: Render app download popup
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~25 minutes
Possibly related PRs
- Moadong/moadong#1551: 대동제 관련 A/B 테스트 및 세션 상태 로직을 Filter, BuskingPage, experiments에서 동시에 정리하는 동일한 범위의 변경입니다.
- Moadong/moadong#1518: 이 PR이 추가한 festivalTimetableNavExperiment와 nav_variant 기반 네비게이션, Filter의 "대동제" 필터 옵션, 세션 기반 NotificationDot 로직을 현재 PR에서 제거합니다.
- Moadong/moadong#1519: 이 PR이 도입한 DAEDONG_POPUP 설정 및 렌더링을 현재 PR에서 APP_DOWNLOAD_POPUP으로 전환하고 webview에서 제거합니다.
Suggested reviewers
- suhyun113
- lepitaaar
✨ 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
develop-fe
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.
🎨 UI 변경사항을 확인해주세요
2개 스토리 변경 · 전체 57개 스토리 · 22개 컴포넌트 |
🚀 릴리즈 PR
📦 버전 정보
💾 BE/💻 FE🚨 MAJOR/➕ MINOR/🔧 PATCHvX.Y.Z📖 버전 라벨 선택 가이드 (Semantic Versioning)
🚨 MAJORv1.0.0→v2.0.0➕ MINORv1.0.0→v1.1.0🔧 PATCHv1.0.0→v1.0.1📋 포함된 변경사항
Summary by CodeRabbit
릴리즈 노트
새로운 기능
버그 수정
개선 사항