-
Notifications
You must be signed in to change notification settings - Fork 0
[FEAT] GTM 설정 #280
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT] GTM 설정 #280
Conversation
WalkthroughGoogle Tag Manager (GTM) integration was added to the HTML page by inserting the GTM script snippet in the Changes
Sequence Diagram(s)sequenceDiagram
participant Browser
participant GTM
Browser->>GTM: Load GTM script asynchronously (from <head>)
alt JavaScript enabled
GTM-->>Browser: Inject GTM functionality
else JavaScript disabled
Browser->>GTM: Load GTM via <noscript> iframe (from <body>)
end
Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
frontend/index.html (2)
15-24: Ensure GTM script is optimally placed and configured
The snippet works but it’s recommended to place it immediately after the opening<head>tag (before other resources) to capture all events. Also, the container ID is hardcoded—consider externalizing it via an env var (e.g.%VITE_GTM_ID%) so you can switch between staging/production without code changes.
28-31: Enhance accessibility for the noscript iframe fallback
The zero-dimension<iframe>is valid, but addingtitle="Google Tag Manager"andaria-hidden="true"will improve accessibility for assistive technologies.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
frontend/index.html(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build-and-test
- GitHub Check: build
Issue Number
close #279
As-Is
To-Be
Check List
Test Screenshot
(Optional) Additional Description
Summary by CodeRabbit