Skip to content

Revert "Feat: GA 세팅 및 연결"#194

Merged
jllee000 merged 1 commit intodevelopfrom
revert-193-feat/#192/GA4-task
Oct 20, 2025
Merged

Revert "Feat: GA 세팅 및 연결"#194
jllee000 merged 1 commit intodevelopfrom
revert-193-feat/#192/GA4-task

Conversation

@jllee000
Copy link
Copy Markdown
Member

@jllee000 jllee000 commented Oct 20, 2025

Reverts #193

Summary by CodeRabbit

릴리스 노트

  • 변경사항

    • 모든 애플리케이션에서 Google Analytics 추적이 제거되었습니다. 페이지 방문, 버튼 클릭, 알림 상호작용 등의 분석 데이터 수집이 중단되었습니다.
  • 개선

    • CI/CD 배포 파이프라인의 캐시 정리 단계가 제거되어 빌드 프로세스가 간소화되었습니다.

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 20, 2025

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

Project Deployment Preview Comments Updated (UTC)
pinback-client-client Ready Ready Preview Comment Oct 20, 2025 7:52am
pinback-client-landing Ready Ready Preview Comment Oct 20, 2025 7:52am

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 20, 2025

Walkthrough

구글 애널리틱스(GA) 추적 기능을 여러 애플리케이션에서 제거하는 변경사항입니다. HTML 파일의 gtag 스크립트, React 컴포넌트의 GA 이벤트 호출, 디자인 시스템의 GA 유틸리티 함수, 서비스 워커의 알림 클릭 핸들러를 삭제하고 Chromatic 캐시 정리 단계를 제거했습니다.

Changes

Cohort / File(s) 변경 요약
GA 스크립트 제거
apps/client/index.html, apps/extension/popup.html, apps/landing/index.html
각 HTML 파일에서 Google Analytics gtag.js 스크립트 태그 및 초기화 코드 블록 삭제
페이지 뷰 추적 제거
apps/client/src/pages/onBoarding/OnBoarding.tsx, apps/client/src/pages/remind/Remind.tsx
useEffect 훅에서 trackPageView 호출 및 import 제거
GA 이벤트 로깅 제거
apps/extension/src/App.tsx, apps/landing/src/components/FinalCTASection.tsx, apps/landing/src/components/Header.tsx
클릭 핸들러에서 GA 이벤트 호출(sendGAEvent) 제거 및 인라인 함수로 변경
GA 유틸리티 함수 제거
packages/design-system/src/components/gtag/utils/gtag.ts, packages/design-system/src/components/index.ts
sendGAEvent 및 trackPageView 함수 정의 및 export 제거
서비스 워커 변경
apps/client/public/firebase-messaging-sw.js
notificationclick 이벤트 리스너 및 GA 이벤트 송신 로직 제거
CI/CD 정리
.github/workflows/chromatic.yml
CHROMATIC_TEMP_DIR 환경변수 및 "Clean storybook cache" 스텝 제거
포맷팅
apps/landing/src/App.tsx
App.css import 후 빈 줄 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

이번 변경사항은 구글 애널리틱스 추적 기능을 일관되게 제거하는 반복적인 패턴입니다. 변경 내용이 동일 유형의 삭제이고 문법적으로 명확하지만, 여러 파일에 걸쳐 있어 완전성 검증이 필요합니다.

Possibly related PRs

Suggested labels

fix, frontend

Suggested reviewers

  • constantly-dev
  • jjangminii

Poem

🐰 애널리틱스를 걷어내고,
추적의 흔적을 지워내며,
깔끔한 코드로 거듭나네!
토끼의 발자국처럼 가볍게,
불필요한 데이터는 안녕! 👋✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning PR 설명은 "Reverts #193"이라는 한 줄로만 구성되어 있으며, 리포지토리의 필수 템플릿을 따르지 않습니다. 템플릿에는 구조화된 섹션 제목(## 📌 Related Issues, ## 📄 Tasks 등)과 명확한 형식이 필요하지만, 제공된 설명에는 이러한 구조가 전혀 없습니다. 되돌리기 작업의 목적, 영향 범위, 또는 검토자를 위한 추가 정보가 부재합니다. 되돌리기 PR이라는 점을 감안하더라도 최소한 템플릿의 기본 구조와 "Related Issues" 섹션 헤더를 따라야 합니다. PR 설명을 리포지토리의 템플릿 형식에 맞게 개선해야 합니다. 최소한 "## 📌 Related Issues" 섹션 헤더를 추가하고 "- close #193" 형식으로 관련 이슈를 명시적으로 태그해야 합니다. 필요한 경우 "## 📄 Tasks" 섹션에서 이 되돌리기가 달성하는 구체적인 목표들을 설명하거나, "## ⭐ PR Point" 섹션에서 검토자를 위한 중요한 정보를 추가할 수 있습니다.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed PR 제목 "Revert "Feat: GA 세팅 및 연결""은 변경 사항과 완벽하게 일치합니다. 원시 요약에 따르면 이 PR은 Google Analytics 통합을 제거하고 있으며, 이는 이전에 추가된 GA 세팅 기능을 되돌리는 것입니다. 제목은 명확하고 구체적이며 리뷰어가 역할을 즉시 이해할 수 있도록 충분한 정보를 제공합니다. "misc updates"와 같은 모호한 용어가 없고 주요 변경 사항을 직접 명시하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-193-feat/#192/GA4-task

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 00bb01b and 267dd1a.

📒 Files selected for processing (13)
  • .github/workflows/chromatic.yml (0 hunks)
  • apps/client/index.html (0 hunks)
  • apps/client/public/firebase-messaging-sw.js (0 hunks)
  • apps/client/src/pages/onBoarding/OnBoarding.tsx (0 hunks)
  • apps/client/src/pages/remind/Remind.tsx (1 hunks)
  • apps/extension/popup.html (0 hunks)
  • apps/extension/src/App.tsx (1 hunks)
  • apps/landing/index.html (0 hunks)
  • apps/landing/src/App.tsx (1 hunks)
  • apps/landing/src/components/FinalCTASection.tsx (2 hunks)
  • apps/landing/src/components/Header.tsx (1 hunks)
  • packages/design-system/src/components/gtag/utils/gtag.ts (0 hunks)
  • packages/design-system/src/components/index.ts (1 hunks)
💤 Files with no reviewable changes (7)
  • packages/design-system/src/components/gtag/utils/gtag.ts
  • apps/client/public/firebase-messaging-sw.js
  • apps/landing/index.html
  • apps/extension/popup.html
  • apps/client/index.html
  • apps/client/src/pages/onBoarding/OnBoarding.tsx
  • .github/workflows/chromatic.yml
🔇 Additional comments (8)
apps/landing/src/App.tsx (1)

2-2: 포맷팅 변경이 확인되었습니다.

순수한 공백 추가로 기능에 영향이 없습니다.

apps/extension/src/App.tsx (1)

26-28: GA 이벤트 로깅이 제거되었습니다.

handleDuplicateRightClick 함수에서 GA 추적 코드가 제거되었고, 탭 생성 기능은 정상적으로 유지됩니다.

apps/landing/src/components/Header.tsx (2)

2-2: GA 유틸리티 import가 제거되었습니다.

sendGAEvent import가 제거되어 Button 컴포넌트만 import합니다.


9-16: 인라인 핸들러로 리팩토링되었습니다.

handleHeaderInstallClick 함수가 제거되고 인라인 onClick 핸들러로 대체되었습니다. GA 추적 로직이 제거되었으나 Chrome Web Store로의 네비게이션 기능은 동일하게 유지됩니다.

apps/landing/src/components/FinalCTASection.tsx (2)

3-3: GA 유틸리티 import가 제거되었습니다.

sendGAEvent import가 제거되어 Button 컴포넌트만 import합니다.


22-30: 인라인 핸들러로 리팩토링되었습니다.

handleInstallClick 함수가 제거되고 인라인 onClick 핸들러로 대체되었습니다. GA 추적 로직이 제거되었으나 Chrome Web Store로의 네비게이션 기능은 동일하게 유지됩니다.

apps/client/src/pages/remind/Remind.tsx (1)

1-2: 페이지 뷰 추적이 제거되었습니다.

trackPageView import가 제거되어 이 페이지에서 GA 페이지 뷰 추적이 비활성화되었습니다. BadgePopupContainer import는 정상적으로 유지됩니다.

packages/design-system/src/components/index.ts (1)

19-19: 검토 의견이 부정확합니다.

코드베이스 전체 검증 결과, sendGAEventtrackPageView 함수는 존재하지 않으며, 어디서도 import되거나 사용되지 않습니다. git 이력에서도 이 함수들이 packages/design-system/src/components/index.ts에서 제거된 기록이 없습니다.

현재 line 19의 export는 GA 유틸이 아닌 ValidateData의 날짜/시간 검증 함수(validateDate, validateTime)입니다. 따라서 언급된 GA 유틸 제거 우려는 실제 코드 변경과 맞지 않습니다.

Likely an incorrect or invalid review comment.


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

✅ Storybook chromatic 배포 확인:
🐿️ storybook

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant