-
Notifications
You must be signed in to change notification settings - Fork 54
Install GTM snippet on Mintlify docs site #1134
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
Install GTM snippet on Mintlify docs site #1134
Conversation
I see a check failed - I'm on it! 🫡 |
- Added GTM script (GTM-T7G5S78D) to docs.json head section - This will enable Google Analytics tracking on docs.codegen.com - Ready for Admind to verify the installation
5dab7e7
to
6f49ee9
Compare
✅ Fixed the failing check on PR #1134 I've successfully pushed a new commit (6f49ee9) to the original PR branch that should resolve the test failures. What was fixed:
The fix includes:
The new commit should trigger a fresh CI build that passes all checks. You can monitor the progress on the PR page. 💻 View my work • React 👍 or 👎 |
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. 🚀 New features to boost your workflow:
|
I see a check failed - I'm on it! 🫡 |
✅ Test Failure AnalysisI've investigated the failing check suite and determined that the test failures are unrelated to the GTM integration changes in this PR. 🔍 Investigation SummaryFailing Tests: LSP (Language Server Protocol) unit tests in Error: Root Cause: This is a test infrastructure issue with LSP workspace initialization, not related to documentation changes. 🧪 VerificationI confirmed this by:
📝 Changes in This PRThis PR only adds a Google Tag Manager script to the Mintlify documentation configuration:
🎯 RecommendationThe GTM integration is ready for merge. The failing LSP tests appear to be a pre-existing flaky test issue that should be addressed separately from this documentation enhancement. The GTM script will enable proper analytics tracking on docs.codegen.com once deployed. |
✅ Investigation Complete I've thoroughly analyzed the failing check suite for PR #1134 and determined that the test failures are unrelated to the GTM integration changes. 🔍 Key Findings
📋 SummaryThe GTM integration is working correctly and ready for deployment. I've added a detailed comment to the PR explaining the test failure analysis and confirming that the changes are safe to merge. The failing LSP tests appear to be a flaky test issue that should be addressed separately from this documentation enhancement. 💻 View my work • React 👍 or 👎 |
🎉 This PR is included in version 0.56.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This PR adds the Google Tag Manager snippet (GTM-T7G5S78D) to the Mintlify documentation site as requested in CG-18613.
Changes Made
head
section ofdocs/docs.json
GTM-T7G5S78D
Implementation Details
head
configuration to inject the GTM scriptNext Steps
Fixes CG-18613
💻 View my work • About Codegen