Skip to content

[hotfix] 상세페이지 알림 버튼 구독 이벤트 트래킹 누락 수정 - #1898

Merged
seongwon030 merged 1 commit into
mainfrom
hotfix/club-detail-subscribe-tracking
Aug 5, 2026
Merged

[hotfix] 상세페이지 알림 버튼 구독 이벤트 트래킹 누락 수정#1898
seongwon030 merged 1 commit into
mainfrom
hotfix/club-detail-subscribe-tracking

Conversation

@seongwon030

Copy link
Copy Markdown
Member

문제

동아리 상세페이지 상단바의 알림(구독) 버튼 클릭이 Mixpanel에 전혀 잡히지 않았다.

메인·구독 목록의 구독 토글은 useWebviewSubscribetoggleSubscribe를 거치고 트래킹도 이 훅 안에 있는데, ClubDetailTopBar는 자체 isNotificationActive 상태를 들고 있어 훅을 쓰지 않고 requestSubscribeToggle(clubId)직접 호출한다. 구독 동작 자체는 정상이지만 트래킹만 우회돼서, Webview Subscribe Toggled 이벤트에 상세페이지 유입분이 통째로 빠져 있었다.

변경

  1. ClubDetailTopBar.handleNotificationClick에 트래킹 추가
  2. 진입 지점을 구분할 수 있도록 Webview Subscribe Toggledsource 프로퍼티 추가
    • toggleSubscribe(clubId, subscribed, source)필수 인자로 둬서 앞으로 새 호출부가 생겨도 source 누락이 타입 에러로 잡힌다
    • 값은 새로 만들지 않고 기존 PAGE_NAME을 재사용 (ClubCard page 프로퍼티·스크롤 트래킹과 같은 체계). CLUB_DETAIL: 'club-detail'만 추가

최종 이벤트 스키마 — club_id, subscribed(토글 후 상태), source

범위

  • 변경 파일 5개, +22 / -5. main 기준 클린 체리픽 (충돌 없음)
  • 동작 변경 없음. 기존 구독 토글 동작은 그대로이고 트래킹 이벤트만 추가된다
  • 같은 내용이 develop-fe 대상으로도 올라가 있다 ([fix] 상세페이지 알림 버튼 구독 이벤트 트래킹 누락 수정 #1897). 릴리즈 대기가 길어 트래킹 공백이 계속 쌓이므로 main에 먼저 반영한다

확인

  • tsc --noEmit 통과 (main 베이스에서 검증)
  • 상세페이지 알림 버튼 클릭 → Webview Subscribe Toggled (source: club-detail) 발생 확인
  • 메인/구독 목록 토글 → 기존과 동일하게 발생, source만 추가 확인

상세페이지 상단바 알림 버튼이 useWebviewSubscribe를 거치지 않고
requestSubscribeToggle을 직접 호출해 구독 토글 이벤트가 누락됐다.

트래킹을 추가하고, 메인·구독 목록과 진입 지점을 구분할 수 있도록
Webview Subscribe Toggled에 source 프로퍼티를 붙인다.
@seongwon030 seongwon030 added 💻 FE Frontend 🛠Fix 기능이 의도한 대로 동작하지 않는 버그를 수정 labels Aug 5, 2026
@seongwon030 seongwon030 self-assigned this Aug 5, 2026
@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
moadong Ready Ready Preview Aug 5, 2026 11:17am

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@seongwon030, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 49 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 0b112cb6-ef22-4195-bfa9-a9888544f8f1

📥 Commits

Reviewing files that changed from the base of the PR and between d0d81fb and b036b66.

📒 Files selected for processing (5)
  • frontend/src/constants/eventName.ts
  • frontend/src/hooks/useWebviewSubscribe.ts
  • frontend/src/pages/ClubDetailPage/components/ClubDetailTopBar/ClubDetailTopBar.tsx
  • frontend/src/pages/MainPage/MainPage.tsx
  • frontend/src/pages/SubscriptionsPage/SubscriptionsPage.tsx

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.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

✅ UI 변경사항 없음

구분 링크
📖 Storybook https://67904e61c16daa99a63b44a7-lzaojgcpfq.chromatic.com/

전체 95개 스토리 · 34개 컴포넌트

@seongwon030 seongwon030 added 📈 release 릴리즈 배포 🔧 PATCH Patch 릴리즈 labels Aug 5, 2026
@seongwon030
seongwon030 merged commit acbd9be into main Aug 5, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend 🛠Fix 기능이 의도한 대로 동작하지 않는 버그를 수정 🔧 PATCH Patch 릴리즈 📈 release 릴리즈 배포

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant