Skip to content

[8222] Add Google Tag Manager - #8225

Merged
n-lark merged 2 commits into
mainfrom
8222-google-tag
Aug 24, 2026
Merged

[8222] Add Google Tag Manager#8225
n-lark merged 2 commits into
mainfrom
8222-google-tag

Conversation

@n-lark

@n-lark n-lark commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Description

See #8222
Test plan #8222 (comment)

Separate issue to add to helm chart is here: FlowFuse/helm#1008

Related Issue(s)

Resolves #8222

Checklist

Labels

  • Includes a DB migration? -> add the area:migration label

@n-lark
n-lark requested review from cstns, hardillb and knolleary August 19, 2026 18:19
@n-lark n-lark self-assigned this Aug 19, 2026
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.19%. Comparing base (4732870) to head (873aa85).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
forge/routes/ui/index.js 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8225      +/-   ##
==========================================
- Coverage   76.20%   76.19%   -0.01%     
==========================================
  Files         443      443              
  Lines       23728    23731       +3     
  Branches     6315     6316       +1     
==========================================
  Hits        18081    18081              
- Misses       5647     5650       +3     
Flag Coverage Δ
backend 76.19% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread docs/install/configuration.md
@cstns

cstns commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Marketing wants Google Tag Manager (container GTM-55L36797) on the free-trial signup page (/account/create).

We're injecting the gtm script globally, we could do it on a per page basis by loading/unloading on page enter/leave.

@n-lark

n-lark commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Hey @cstns went global because of marketing. They can target pages themselves with a Page Path trigger, and I figured they'd want it on more pages down the line, so global was the flexible setup. We can scope it to /account/create if we'd rather, just note it'll never unmount after you hit that page without a refresh.

@n-lark
n-lark merged commit fbe55c1 into main Aug 24, 2026
31 of 32 checks passed
@n-lark
n-lark deleted the 8222-google-tag branch August 24, 2026 17:41
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.

Add Google Tag Manager

3 participants