chore: Google Tag Manager(GTM) 도입 및 기존 GA4 태그 제거#49
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughGoogle Analytics를 Google Tag Manager로 마이그레이션하는 변경으로, GoogleTagManager 컴포넌트를 import하여 HTML 루트에 GTM ID와 함께 렌더링하고 GoogleAnalytics 컴포넌트 사용을 제거했습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@app/layout.tsx`:
- Line 9: 현재 app/layout.tsx에서 import 구문에 사용되지 않는 심볼 GoogleAnalytics가 포함되어 있습니다;
import 문에서 GoogleAnalytics를 제거하고 GoogleTagManager만 남기도록 수정하세요 (즉, import {
GoogleTagManager } from '@next/third-parties/google';로 정리), 변경 후 빌드나 lint 에러가
없는지 확인하세요.
| import GlobalModal from '@/components/modal/globalModal'; | ||
| import QueryProvider from '@/components/providers/queryProvider'; | ||
| import { GoogleAnalytics } from '@next/third-parties/google'; | ||
| import { GoogleAnalytics, GoogleTagManager } from '@next/third-parties/google'; |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
cat -n app/layout.tsxRepository: SWYP-mingling/Frontend
Length of output: 2555
🏁 Script executed:
rg -n '\bGoogleAnalytics\b' app/layout.tsxRepository: SWYP-mingling/Frontend
Length of output: 147
사용하지 않는 GoogleAnalytics import를 제거해주세요.
Line 9에서 GoogleAnalytics를 import했지만 파일에서 사용되지 않습니다. GoogleTagManager만 필요하므로 import를 정정하면 됩니다.
-import { GoogleAnalytics, GoogleTagManager } from '@next/third-parties/google';
+import { GoogleTagManager } from '@next/third-parties/google';📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| import { GoogleAnalytics, GoogleTagManager } from '@next/third-parties/google'; | |
| import { GoogleTagManager } from '@next/third-parties/google'; |
🧰 Tools
🪛 GitHub Check: build
[warning] 9-9:
'GoogleAnalytics' is defined but never used
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@app/layout.tsx` at line 9, 현재 app/layout.tsx에서 import 구문에 사용되지 않는 심볼
GoogleAnalytics가 포함되어 있습니다; import 문에서 GoogleAnalytics를 제거하고 GoogleTagManager만
남기도록 수정하세요 (즉, import { GoogleTagManager } from '@next/third-parties/google';로
정리), 변경 후 빌드나 lint 에러가 없는지 확인하세요.
2 participants
🚀chore: Google Tag Manager(GTM) 도입 및 기존 GA4 태그 제거
📝 변경사항
✅ 체크리스트
📸 스크린샷
💬 리뷰어 전달사항
Summary by CodeRabbit