[feature] 대동제 종료 — A/B 테스트 정리 및 관련 코드 제거 by seongwon030 · Pull Request #1551 · Moadong/moadong

seongwon030 · 2026-05-14T14:49:28Z

#️⃣연관된 이슈

ex) #1550

📝작업 내용

배경

2026 대동제 버스킹 시간표 페이지에서 진행한 festival_timetable_nav_v1 A/B 테스트가 종료됨.
tabs 변형이 인당 날짜 탐색 횟수에서 우위를 보여 tabs UI로 고정하고, 대동제 관련 임시 코드 전체를 제거함.

실험 결과 요약

  ┌─────────┬─────────┬───────────┬───────────┬────────────┐
  │ variant │ 유저 수 │ 총 이벤트 │ 인당 평균 │ swipe 비율 │
  ├─────────┼─────────┼───────────┼───────────┼────────────┤
  │ tabs    │ 73명    │ 514       │ 7.0회     │ 23.3%      │
  ├─────────┼─────────┼───────────┼───────────┼────────────┤
  │ arrows  │ 61명    │ 350       │ 5.7회     │ 20.3%      │
  └─────────┴─────────┴───────────┴───────────┴────────────┘

→ tabs 승리 (인당 탐색 23% 우위), tabs UI로 고정

변경 내용

실험 코드 제거

src/experiments/definitions.ts — festivalTimetableNavExperiment
제거, ALL_EXPERIMENTS = []
src/pages/FestivalPage/BuskingPage/BuskingPage.tsx — DayTabsNav로
고정, DayArrowsNav / useExperimentVariant / nav_variant 프로퍼티 제거

라우트 및 필터 제거

src/routes/AppRoutes.tsx — /festival-busking, /festival-introduction
라우트 제거
src/routes/webviewRoutes.tsx — /webview/festival-busking 라우트 제거
src/routes/webviewFilterConfig.ts — 대동제 탭 제거
src/components/common/Filter/Filter.tsx — 대동제 칩, daedongDotSeen
세션스토리지 로직 제거

팝업 복구

src/pages/MainPage/MainPage.tsx — DAEDONG_POPUP → APP_DOWNLOAD_POPUP
복구
src/pages/WebviewMainPage/WebviewMainPage.tsx — 팝업 제거 (원래 없던
상태로 복원)
src/pages/MainPage/components/Popup/popupConfigs.ts — DAEDONG_POPUP
및 daedong 이미지 제거

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

Release Notes

New Features
- 앱 다운로드 팝업 추가
Refactor
- 축제 및 대동제 관련 기능 제거
- 실험 기반 네비게이션 코드 제거
- 필터 옵션 및 로직 단순화

- festivalTimetableNavExperiment 실험 제거, tabs UI로 고정 - 대동제 필터 칩 및 /festival-busking, /festival-introduction 라우트 제거 - Filter.tsx의 daedongDotSeen 세션스토리지 로직 제거

- MainPage: DAEDONG_POPUP → APP_DOWNLOAD_POPUP 복구 - WebviewMainPage: 팝업 제거 (원래 없던 상태로 복원) - popupConfigs: DAEDONG_POPUP 및 daedong 이미지 import 제거

vercel · 2026-05-14T14:49:34Z

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

Project	Deployment	Actions	Updated (UTC)
moadong	Ready	Preview, Comment	May 14, 2026 2:49pm

coderabbitai · 2026-05-14T14:49:41Z

Warning

`.coderabbit.yaml` has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)

Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"

⚙️ Configuration instructions

Please see the configuration documentation for more information.
You can also validate your configuration using the online YAML validator.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

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: 13a5fe68-d162-4180-8bec-3446064156b8

📥 Commits

Reviewing files that changed from the base of the PR and between ff0802b and b7eb1fd.

📒 Files selected for processing (9)

frontend/src/components/common/Filter/Filter.tsx
frontend/src/experiments/definitions.ts
frontend/src/pages/FestivalPage/BuskingPage/BuskingPage.tsx
frontend/src/pages/MainPage/MainPage.tsx
frontend/src/pages/MainPage/components/Popup/popupConfigs.ts
frontend/src/pages/WebviewMainPage/WebviewMainPage.tsx
frontend/src/routes/AppRoutes.tsx
frontend/src/routes/webviewFilterConfig.ts
frontend/src/routes/webviewRoutes.tsx

💤 Files with no reviewable changes (5)

frontend/src/routes/webviewFilterConfig.ts
frontend/src/routes/webviewRoutes.tsx
frontend/src/pages/WebviewMainPage/WebviewMainPage.tsx
frontend/src/pages/MainPage/components/Popup/popupConfigs.ts
frontend/src/routes/AppRoutes.tsx

Walkthrough

This PR removes all festival-related features from the frontend, including filter options, navigation experiments, and routes. The festival popup is replaced with an app download popup on the main page, and the webview popup is removed entirely. Navigation in BuskingPage is unified to always show the day tabs interface.

Changes

Festival 기능 제거 및 앱 다운로드 팝업 통합

Layer / File(s)	Summary
Filter 컴포넌트: 대동제 옵션 제거 및 알림 도트 단순화 `frontend/src/components/common/Filter/Filter.tsx`	`WEB_FILTER_OPTIONS`에서 대동제 옵션 항목 제거, `useState` 및 `sessionStorage` 기반 상태 로직 삭제, `handleFilterOptionClick` 단순화로 클릭 트래킹만 수행, `NotificationDot` 조건을 홍보 라벨 + `hasNotification`으로 단일화.
BuskingPage: 실험 기반 네비게이션 제거 및 DayTabsNav로 통일 `frontend/src/experiments/definitions.ts`, `frontend/src/pages/FestivalPage/BuskingPage/BuskingPage.tsx`	실험 정의 `ALL_EXPERIMENTS`를 빈 배열로 변경, `festivalTimetableNavExperiment` 제거. BuskingPage에서 실험 기반 네비게이션 분기 제거, `Styled.NavWrapper`에서 `DayTabsNav`만 렌더링하도록 통일. `DAY_DURATION`/`DAY_CHANGED` 트래킹 이벤트에서 `nav_variant` 필드 제거.
팝업 설정 변경: DAEDONG_POPUP → APP_DOWNLOAD_POPUP `frontend/src/pages/MainPage/components/Popup/popupConfigs.ts`, `frontend/src/pages/MainPage/MainPage.tsx`, `frontend/src/pages/WebviewMainPage/WebviewMainPage.tsx`	`APP_DOWNLOAD_POPUP` 설정 추가로 앱 다운로드 이미지 사용, 클릭 시 `USER_EVENT.APP_DOWNLOAD_POPUP_CLICKED` 트래킹 및 앱 스토어 링크 오픈. MainPage의 팝업을 `APP_DOWNLOAD_POPUP`으로 교체. WebviewMainPage에서 팝업 import/렌더링 제거.
라우팅 정리: 페스티벌 경로 제거 `frontend/src/routes/AppRoutes.tsx`, `frontend/src/routes/webviewFilterConfig.ts`, `frontend/src/routes/webviewRoutes.tsx`	`AppRoutes`에서 `/festival-introduction`, `/festival-busking` 경로 및 대응하는 컴포넌트 import 제거. `WEBVIEW_FILTER_CONFIG`에서 대동제 필터 항목 제거. `webviewRoutes`에서 `BuskingPage` import 및 `/webview/festival-busking` 매핑 제거.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Moadong/moadong#1010: MainPage의 APP_DOWNLOAD_POPUP 팝업 설정 변경과 사용자 이벤트 트래킹 구현이 직접 관련됨.
Moadong/moadong#1518: Filter의 대동제 옵션/NotificationDot 로직 제거와 festivalTimetableNavExperiment 기반 BuskingPage 라우팅 제거가 직접적으로 맞물림.
Moadong/moadong#1536: Filter.tsx의 웹뷰 필터 옵션 및 WEBVIEW_FILTER_CONFIG 참조 로직 변경과 동일 영역에서 맞물림.

Suggested reviewers

suhyun113
oesnuj
lepitaaar

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 목적을 명확하게 나타냅니다. 대동제 A/B 테스트 정리 및 관련 코드 제거라는 핵심 내용을 간결하고 구체적으로 전달합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ 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 feature/#1550-cleanup-festival-daedong-MOA-858

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

github-actions · 2026-05-14T14:50:52Z

🎨 UI 변경사항을 확인해주세요

변경된 스토리를 Chromatic에서 확인해주세요.

구분	링크
🔍 변경사항 리뷰	https://www.chromatic.com/build?appId=67904e61c16daa99a63b44a7&number=333
📖 Storybook	https://67904e61c16daa99a63b44a7-bxlaippgpr.chromatic.com/

4개 스토리 변경 · 전체 57개 스토리 · 22개 컴포넌트

seongwon030 added 2 commits May 14, 2026 23:43

refactor(main): 대동제 팝업 제거 및 앱 다운로드 팝업 복구

b7eb1fd

- MainPage: DAEDONG_POPUP → APP_DOWNLOAD_POPUP 복구 - WebviewMainPage: 팝업 제거 (원래 없던 상태로 복원) - popupConfigs: DAEDONG_POPUP 및 daedong 이미지 import 제거

seongwon030 self-assigned this May 14, 2026

seongwon030 added ✨ Feature 기능 개발 💻 FE Frontend AB TEST AB 테스트 관련 labels May 14, 2026

seongwon030 requested a review from suhyun113 May 14, 2026 14:50

seongwon030 merged commit 2c9e00b into develop-fe May 15, 2026
5 checks passed

seongwon030 deleted the feature/#1550-cleanup-festival-daedong-MOA-858 branch May 15, 2026 04:42

coderabbitai Bot mentioned this pull request May 15, 2026

[release] FE v1.7.3 #1552

Merged

coderabbitai Bot mentioned this pull request May 31, 2026

[feature] 대동제 페이지를 홍보 게시판으로 마이그레이션한다 #1619

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feature] 대동제 종료 — A/B 테스트 정리 및 관련 코드 제거#1551

[feature] 대동제 종료 — A/B 테스트 정리 및 관련 코드 제거#1551
seongwon030 merged 2 commits into
develop-fefrom
feature/#1550-cleanup-festival-daedong-MOA-858

seongwon030 commented May 14, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

vercel Bot commented May 14, 2026

Uh oh!

coderabbitai Bot commented May 14, 2026 •

edited

Loading

`.coderabbit.yaml` has a parsing error

Uh oh!

github-actions Bot commented May 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

seongwon030 commented May 14, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

#️⃣연관된 이슈

📝작업 내용

배경

변경 내용

실험 코드 제거

라우트 및 필터 제거

팝업 복구

중점적으로 리뷰받고 싶은 부분(선택)

논의하고 싶은 부분(선택)

🫡 참고사항

Summary by CodeRabbit

Release Notes

Uh oh!

vercel Bot commented May 14, 2026

Uh oh!

coderabbitai Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

.coderabbit.yaml has a parsing error

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

github-actions Bot commented May 14, 2026

🎨 UI 변경사항을 확인해주세요

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

seongwon030 commented May 14, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented May 14, 2026 •

edited

Loading

`.coderabbit.yaml` has a parsing error