Skip to content

Conversation

@dleffel
Copy link
Collaborator

@dleffel dleffel commented Jun 29, 2025

Summary
Implements Google Tag Manager (GTM) on the Roo Code marketing website to enable analytics and tracking capabilities.

Changes
Added GTM script tag in the section of apps/web-roo-code/src/app/layout.tsx as high as possible per Google's specifications
Added GTM noscript fallback immediately after the opening tag for users with JavaScript disabled
Used container ID GTM-M2JZHV8N as specified in the implementation requirements
Implemented in root layout to ensure site-wide coverage across all pages
Technical Details
Follows Google's official GTM implementation guidelines
Uses React's dangerouslySetInnerHTML for script injection (Next.js best practice)
Proper inline styles for noscript iframe element
Verified working across multiple pages (home, enterprise, privacy)
Testing
✅ Verified GTM tags render correctly on:

Home page (/)
Enterprise page (/enterprise)
Privacy page (/privacy)
Both the main GTM script and noscript fallback are properly implemented and functional.


Important

Add Google Tag Manager to layout.tsx for site-wide analytics and tracking.

  • Behavior:
    • Adds Google Tag Manager script in <head> and noscript fallback in <body> in layout.tsx.
    • Uses container ID GTM-M2JZHV8N for tracking.
    • Ensures site-wide coverage by implementing in root layout.
  • Technical Details:
    • Follows Google's GTM implementation guidelines.
    • Uses dangerouslySetInnerHTML for script injection.
    • Proper inline styles for noscript iframe.
  • Testing:
    • Verified GTM tags render correctly on home, enterprise, and privacy pages.

This description was created by Ellipsis for 7846e51. You can customize this summary. It will automatically update as commits are pushed.

- Add GTM script tag in head section as high as possible
- Add GTM noscript fallback immediately after body opening tag
- Use container ID GTM-M2JZHV8N as specified
- Implement in root layout for site-wide coverage
- Follow Google's implementation guidelines and Next.js best practices
@dleffel dleffel requested review from cte, jr and mrubens as code owners June 29, 2025 17:23
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 29, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 29, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Jun 30, 2025
@hannesrudolph hannesrudolph added PR - Needs Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 30, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 30, 2025
@mrubens
Copy link
Collaborator

mrubens commented Jul 3, 2025

We discussed a bit - going to move this to draft for now

@mrubens mrubens marked this pull request as draft July 3, 2025 03:49
@daniel-lxs daniel-lxs moved this from PR [Needs Review] to PR [Draft / In Progress] in Roo Code Roadmap Jul 30, 2025
@github-project-automation github-project-automation bot moved this from PR [Draft / In Progress] to Done in Roo Code Roadmap Sep 23, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Draft / In Progress size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants