Skip to content

Support for GA4 Tag Manager - #350

Merged
levgorbunov1 merged 3 commits into
mainfrom
ga4-tag-manager
Aug 5, 2025
Merged

Support for GA4 Tag Manager#350
levgorbunov1 merged 3 commits into
mainfrom
ga4-tag-manager

Conversation

@heathd

@heathd heathd commented May 9, 2024

Copy link
Copy Markdown
Contributor

adding support for GA4 tag manager. Can activate by setting ga4_tracking_id config variable.

I have not reviewed the custom analytics tracking javascript relating to search operations, so that may not work

What’s changed

Add support for GA4 tag manager

Identifying a user need

The GDS Engineering Enablement team need this as we are using TDT for our team documentation and would like to add google analytics support

@heathd
heathd force-pushed the ga4-tag-manager branch from 53dfb2c to b4afd67 Compare May 9, 2024 10:01
@huwd

huwd commented Oct 9, 2024

Copy link
Copy Markdown
Contributor

Let's look at this for 4.1 there's a GTM PR below too would like to consider the two at the same time

@huwd

huwd commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

@heathd you still interested in this?
I think I shelved reviewing this so we could prioritise some WCAG stuff, I meant to come back earlier but this is the first time I've returned.

Can do a quick bit of testing and get this merged for you?

@davidread

Copy link
Copy Markdown

@huwd Yes please, could you review and merge this? We're currently maintaining a fork until this is done.

Google Universal Analytics is dead, which drives the need for supporting GA4, to avoid going backwards

adding support for GA4 tag manager. Can activate by setting `ga4_tracking_id` config variable.

Have not reviewed the custom analytics tracking javascript relating to search operations

Co-authored-by: Hauwa Muhammad  <HauwaHakimi@users.noreply.github.com>
@huwd
huwd force-pushed the ga4-tag-manager branch from b4afd67 to 2458580 Compare July 29, 2025 14:50
@huwd

huwd commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

I've rebased it against main,
Will check the test suite now

@huwd

huwd commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

Yeah looks reasonable and additive.

@huwd

huwd commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

@huwd
huwd force-pushed the ga4-tag-manager branch from ca1df7f to bfec239 Compare July 29, 2025 23:44
@natf-gds

natf-gds commented Aug 4, 2025

Copy link
Copy Markdown

This can't pass tests until Huw's changes are in #426

426 is replaced by #427

@levgorbunov1
levgorbunov1 merged commit 6d81523 into main Aug 5, 2025
6 of 15 checks passed
@levgorbunov1
levgorbunov1 deleted the ga4-tag-manager branch August 5, 2025 09:24
@davidr-gds

Copy link
Copy Markdown

🎉

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.

6 participants