Skip to content

feat: use Google Tag Manager gtm.js instead of gtag.js - #189

Merged
fvsch merged 3 commits into
mainfrom
fvsch/update-gtm
Dec 15, 2023
Merged

feat: use Google Tag Manager gtm.js instead of gtag.js#189
fvsch merged 3 commits into
mainfrom
fvsch/update-gtm

Conversation

@fvsch

@fvschfvsch commented Dec 14, 2023

Copy link
Copy Markdown
Contributor

Fixes MKT-69.

This PR updates the analytics snippet code to load both /gtm.js and /gtag/js.

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz CodeflowRun & review this pull request in StackBlitz Codeflow.

@stackblitz-staging

Copy link
Copy Markdown

Review PR in StackBlitz CodeflowRun & review this pull request in StackBlitz Codeflow.

@netlify

netlifyBot commented Dec 14, 2023

Copy link
Copy Markdown

Deploy Preview for stackblitz-docs ready!

NameLink
🔨 Latest commite694b89
🔍 Latest deploy loghttps://app.netlify.com/sites/stackblitz-docs/deploys/657c27788f09710008474af0
😎 Deploy Previewhttps://deploy-preview-189--stackblitz-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment thread.vitepress/config.ts

const BASE = '/';
const BASE_WITH_ORIGIN = `https://developer.stackblitz.com${BASE}`;
const BASE_PATH = '/';

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

💅 Unrelated cosmetic changes: renamed a constant, and tweaked formatting (running Prettier).

@peter-zawistowiczpeter-zawistowicz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ 🙏

@fvsch
fvsch merged commit e21a80d into mainDec 15, 2023
@fvsch
fvsch deleted the fvsch/update-gtm branch December 15, 2023 16:06
Sign up for freeto 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.

2 participants

@fvsch@peter-zawistowicz