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

Remove GTM, GA, and the cookie banner #4806

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

Nitemaeric
Copy link
Contributor

@Nitemaeric Nitemaeric commented Jan 8, 2025

Context

We haven't used Google Tag Manager, Google Analytics, and other cookie-based analytics systems for over 12 months. We should remove them from the services.

Changes proposed in this pull request

  • Remove references to Google Tag Manager and Google Analytics.
  • Remove the cookie banner. Based on the guidance here, we do not need to show a cookie banner.
  • Remove ability to customise cookie preferences.

Guidance to review

A lot of code has been removed, please check to see that nothing is breaking.

@Nitemaeric Nitemaeric added the deploy A Review App will be created for PRs with this label label Jan 8, 2025
@Nitemaeric Nitemaeric self-assigned this Jan 8, 2025
@Nitemaeric Nitemaeric force-pushed the dd/remove-gtm-ga-and-cookie-banner branch from eeb3e96 to 77fca36 Compare January 8, 2025 15:18
@Nitemaeric Nitemaeric marked this pull request as ready for review January 8, 2025 15:58
@Nitemaeric Nitemaeric requested a review from a team as a code owner January 8, 2025 15:58
Copy link
Contributor

@gms-gs gms-gs left a comment

Choose a reason for hiding this comment

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

NOICE

@Nitemaeric Nitemaeric force-pushed the dd/remove-gtm-ga-and-cookie-banner branch from 77fca36 to 233640c Compare January 9, 2025 15:24
@Nitemaeric Nitemaeric merged commit 302ee03 into main Jan 9, 2025
19 checks passed
@Nitemaeric Nitemaeric deleted the dd/remove-gtm-ga-and-cookie-banner branch January 9, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy A Review App will be created for PRs with this label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants