Skip to content

Conversation

reakaleek
Copy link
Member

No description provided.

@reakaleek reakaleek requested a review from a team as a code owner March 25, 2025 22:23
@reakaleek reakaleek self-assigned this Mar 25, 2025
google_tag_manager:
enabled: true
script: |
<script>(function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
Copy link
Member

Choose a reason for hiding this comment

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

Can we break this up into the actual variable bits?

It will be more tedious to configure but since the navigation.yml file might end up living in docs-content I am weary of raw including HTML from "user" input.

Copy link
Member

Choose a reason for hiding this comment

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

assembler.yml too 😸

Copy link
Member Author

Choose a reason for hiding this comment

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

SGTM.

24b955f

@reakaleek reakaleek requested a review from Mpdreamz March 26, 2025 09:37
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM!

@reakaleek reakaleek merged commit d5e7c9b into main Mar 26, 2025
11 checks passed
@reakaleek reakaleek deleted the feature/prod-google-tag-manager branch March 26, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants