Skip to content

[release] FE v1.7.3#1552

Merged
seongwon030 merged 3 commits into
mainfrom
develop-fe
May 15, 2026
Merged

[release] FE v1.7.3#1552
seongwon030 merged 3 commits into
mainfrom
develop-fe

Conversation

@seongwon030
Copy link
Copy Markdown
Member

@seongwon030 seongwon030 commented May 15, 2026

🚀 릴리즈 PR

📦 버전 정보

항목 내용
서비스 💾 BE / 💻 FE
Bump 타입 🚨 MAJOR / ➕ MINOR / 🔧 PATCH
예상 버전 vX.Y.Z

⚠️ 반드시 라벨을 지정해주세요: 서비스 라벨(💾 BE, 💻 FE)과 버전 라벨(🚨 MAJOR, ➕ MINOR, 🔧 PATCH)이 없으면 태그가 생성되지 않습니다.

📖 버전 라벨 선택 가이드 (Semantic Versioning)
라벨 버전 변화 선택 기준 예시
🚨 MAJOR v1.0.0v2.0.0 기존 API/기능이 호환되지 않는 변경 API 엔드포인트 삭제/변경, 요청/응답 스펙 변경, DB 스키마 대규모 변경
➕ MINOR v1.0.0v1.1.0 기존 기능은 유지하면서 새 기능 추가 새 API 엔드포인트 추가, 새 기능 도입, 기존 API에 선택적 필드 추가
🔧 PATCH v1.0.0v1.0.1 기능 변경 없이 버그 수정/내부 개선 버그 수정, 성능 개선, 리팩토링, 문서 수정

📋 포함된 변경사항

이번 릴리즈에 포함된 주요 변경사항을 요약합니다.

Summary by CodeRabbit

릴리즈 노트

  • 새로운 기능

    • 앱 다운로드 팝업이 이제 메인 페이지에 표시됩니다.
  • 버그 수정

    • 날짜 네비게이션이 항상 일관되게 표시되도록 개선되었습니다.
  • 개선 사항

    • 필터 및 라우팅이 정리되어 앱 성능이 최적화되었습니다.

Review Change Stack

- 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 테스트 정리 및 관련 코드 제거
@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

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

Project Deployment Actions Updated (UTC)
moadong Ready Ready Preview, Comment May 15, 2026 4:43am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

Caution

Review failed

The pull request is closed.

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
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f73728a-5f80-4ce8-980e-17038ffac72c

📥 Commits

Reviewing files that changed from the base of the PR and between c0a47b7 and 2c9e00b.

📒 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

Disabled knowledge base sources:

  • Jira integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

이 PR은 "대동제" 페스티벌 기능 전체를 제거합니다. 실험 정의를 비우고, BuskingPage의 조건부 네비게이션을 단순화하며, Filter의 세션 상태 로직을 정리하고, 메인 페이지 팝업을 DAEDONG_POPUP에서 APP_DOWNLOAD_POPUP으로 전환합니다.

Changes

Festival Feature Removal and UI Consolidation

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)
Loading
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
Loading

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

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

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

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

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

@seongwon030 seongwon030 merged commit f6a927e into main May 15, 2026
4 of 5 checks passed
@seongwon030 seongwon030 changed the title Develop fe [release] FE v1.7.3 May 15, 2026
@coderabbitai coderabbitai Bot mentioned this pull request May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend 🔧 PATCH Patch 릴리즈 📈 release 릴리즈 배포

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants