Skip to content

[8222] Add Google Tag Manager - #8225

Open
n-lark wants to merge 1 commit into
mainfrom
8222-google-tag
Open

[8222] Add Google Tag Manager#8225
n-lark wants to merge 1 commit 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

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
  • Link to Changelog Entry PR, or note why one is not needed.

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.18%. Comparing base (d33b229) to head (1eef290).

Files with missing lines Patch % Lines
forge/routes/ui/index.js 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    FlowFuse/flowfuse#8225      +/-   ##
==========================================
- Coverage   76.19%   76.18%   -0.01%     
==========================================
  Files         443      443              
  Lines       23726    23729       +3     
  Branches     6314     6315       +1     
==========================================
  Hits        18079    18079              
- Misses       5647     5650       +3     
Flag Coverage Δ
backend 76.18% <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.

`telemetry.frontend.google.tag` | A Google Analytics (gtag) measurement ID. Default: `null`
`telemetry.frontend.google.events` | An object with keys matching the names of tag events to be enabled and any payload values. Default `null`

`telemetry.frontend.google.gtm` | A Google Tag Manager container ID (e.g. `GTM-XXXXXXX`). Loads GTM once cookie consent is accepted. Default: `null`

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Need to do this in AM - only had helm chart issue raised

Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

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

2 participants