-
Notifications
You must be signed in to change notification settings - Fork 2
Remove all Google Tag Manager, cookies, UTM tracking, analytics, and tracking consent code from the website #248
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for testcontainers-site ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@copilot Also remove all cookies from the website. |
Co-authored-by: kiview <[email protected]>
Co-authored-by: kiview <[email protected]>
Done. Removed all cookie and localStorage usage from the website including:
Commit: e6c7a87 |
Co-authored-by: kiview <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of additional cleanups could be done, I think the removing the now redundant iubenda code in the footer layout should be done. The others are more optional.
Other changes seem fine.
README.md
Outdated
|
|
||
| ## Analytics | ||
|
|
||
| The site uses [iubenda](https://www.iubenda.com/) to manage tracking consent. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With cookies removed we can remove this line and all of the iubenda code in the footer
| <div class="subfooter"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Apply this suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed iubenda line from README.md and all iubenda code from footer.html including:
- Privacy Policy and Cookie Policy links
- Your Privacy Choices link with CCPA icon
- Notice at Collection link
- iubenda.js script loader
Commit: 8e69c47
README.md
Outdated
|
|
||
| Tracking scripts are loaded via [Google Tag Manager](https://tagmanager.google.com/#/container/accounts/6068072463/containers/116206463/workspaces/9) | ||
|
|
||
| Analytics are recorded in [Google Analytics](https://analytics.google.com/analytics/web/#/p373834203/reports/home) and [Heap](https://heapanalytics.com/app/env/4204607514/dashboard/Testcontainers-Overview-278374) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Analytics will no longer be recorded. If we want to keep this a link to historical data we can update this line to explain that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Apply this suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the analytics line in README.md to clarify that it links to historical data only, with a note that analytics are no longer being recorded.
Commit: 5a8b6f3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With all of the client side reason for this page removed, we could add it to the redirects in netlify.toml. We may also wan't to update the TCC code to not redirect users to this page anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Apply this suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added redirect in netlify.toml to redirect /desktop/authorized/ directly to https://app.testcontainers.cloud/api/auth/login with a 302 status code. This eliminates the need for the client-side redirect page.
Commit: 5a8b6f3
Co-authored-by: kiview <[email protected]>
Co-authored-by: kiview <[email protected]>
This PR removes all Google Tag Manager (GTM) code, cookies/localStorage, UTM tracking, Mixpanel analytics, and iubenda tracking consent management from the testcontainers-site repository to create a privacy-focused website without any tracking or analytics.
Changes
Removed GTM Scripts
layouts/_default/baseof.htmlhead sectionRemoved All Cookies and Client-Side Storage
getCookiefunctions from bothassets/js/main.jsandlayouts/authorized/baseof.htmlRemoved All UTM Tracking
assets/js/main.jsthat captured referrer and URL parametersparseUtms()function that extracted UTM parameters (utm_campaign, utm_source, utm_medium, utm_term, utm_content)Removed All Mixpanel and Attribution Code
In
layouts/authorized/baseof.html:Removed Tracking Consent Management
In
layouts/partials/footer.html:Added Infrastructure Improvements
In
netlify.toml:/desktop/authorized/tohttps://app.testcontainers.cloud/api/auth/login(302 status)Updated Documentation
Updated
README.mdto:Impact
✅ Privacy-Focused: Website now operates completely without any tracking, cookies, or client-side storage (excluding third-party libraries)
✅ Simpler Code: Removed all analytics and tracking dependencies
✅ Cleaner Experience: Users are no longer tracked across the site
✅ Faster Page Loads: Removed external analytics scripts
✅ No Consent UI: Removed tracking consent management UI from footer
✅ Better Infrastructure: Desktop authorization redirect now handled at CDN level instead of client-side
Verification
Comprehensive search confirms zero remaining references to:
document.cookieorgetCookiefunctionslocalStorageusage (in owned code)Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.