Skip to content

docs: align GTM noscript container ID - #2780

Merged
iwoplaza merged 1 commit into
software-mansion:releasefrom
p-malecki:@p-malecki/update-gtm-id
Aug 3, 2026
Merged

iwoplaza merged 1 commit into
software-mansion:releasefrom
p-malecki:@p-malecki/update-gtm-id

Conversation

@p-malecki

@p-malecki p-malecki commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Unify GTM IDs in apps/typegpu-docs/src/layouts/PageLayout.astro to GTM-T69W3PKG.

Resolves #2777

Copilot AI review requested due to automatic review settings August 3, 2026 08:37

Copilot AI 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.

Pull request overview

This PR fixes an inconsistency in the docs site’s Google Tag Manager (GTM) integration by aligning the <noscript> fallback iframe container ID with the main GTM loader container ID, ensuring JS and no-JS page views are attributed to the same GTM container.

Changes:

  • Updated the GTM <noscript> iframe src to use GTM-T69W3PKG (matching the existing JS loader snippet).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pullfrog pullfrog Bot 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.

✅ No new issues found.

Reviewed changes

  • Align GTM noscript container ID — changed <noscript> iframe src from GTM-TFGF383S to GTM-T69W3PKG to match the JS loader snippet.

Pullfrog  | View workflow run | Using DeepSeek Pro (free via Pullfrog for OSS) (Kimi K2 not used — the program covers this model; add its provider key to run your pick) | 𝕏

@iwoplaza
iwoplaza changed the base branch from main to release August 3, 2026 09:50
@iwoplaza
iwoplaza force-pushed the @p-malecki/update-gtm-id branch from 982d538 to 6956461 Compare August 3, 2026 09:52
@iwoplaza
iwoplaza merged commit 817d559 into software-mansion:release Aug 3, 2026
4 of 5 checks passed
@p-malecki
p-malecki deleted the @p-malecki/update-gtm-id branch August 3, 2026 13:33
Sign up for free to 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.

docs: Google Tag Manager IDs seem to be mismatched

3 participants