Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change GA4 type on service manual contents list #2944

Merged
merged 1 commit into from
Oct 5, 2023
Merged

Conversation

andysellick
Copy link
Contributor

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

What

Change the GA4 type on the contents list on pages like https://www.gov.uk/service-manual/helping-people-to-use-your-service/making-your-service-accessible-an-introduction#make-non-digital-parts-of-your-service-accessible to 'select content'.

Why

These are in-page navigation events.

Visual changes

None.

Trello card: https://trello.com/c/kPQBqdSw/675-change-contents-navigation-selectcontent-event-types-to-contents-list

@andysellick andysellick requested a review from AshGDS October 3, 2023 15:24
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-2944 October 3, 2023 15:24 Inactive
@@ -69,7 +69,7 @@
data-ga4-set-indexes
data-ga4-link="<%= {
event_name: "navigation",
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @andysellick - isn't event_name meant to be select_content, and the type contents list?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Checked with Harriet and it should be event_name = select_content and type = contents list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So exactly what you said 😁

@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-2944 October 5, 2023 10:28 Inactive
Copy link
Contributor

@AshGDS AshGDS left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@andysellick andysellick merged commit 55d852f into main Oct 5, 2023
6 checks passed
@andysellick andysellick deleted the ga4-adjust-sm branch October 5, 2023 11:43
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.

3 participants