Skip to content

Commit

Permalink
Remove google analytics code from 404 / 410 pages
Browse files Browse the repository at this point in the history
UA is going to be decommissioned imminently, so these aren't going to
work shortly.

If we need page view information for bouncer, we can get it from the CDN
logs.

Removing analytics rather than replacing it with GA4 means we don't have
to solve a cookie consent problem.
  • Loading branch information
richardTowers committed Jun 26, 2024
1 parent 0570c3c commit e3ad5e8
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 13 deletions.
7 changes: 0 additions & 7 deletions public/google-analytics.js

This file was deleted.

3 changes: 0 additions & 3 deletions templates/404.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@
<title>404 - Not Found</title>
<link href="/gone.css" media="screen" rel="stylesheet" type="text/css">
<!--[if lte IE 8]><link href="/ie.css" media="screen" rel="stylesheet" type="text/css"><![endif]-->
<% if ENV['RACK_ENV'] != 'development' %>
<script src="/google-analytics.js" type="text/javascript"></script>
<% end %>
</head>
<body>
<div id="wrapper">
Expand Down
3 changes: 0 additions & 3 deletions templates/410.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@
<title>410 - Page Archived</title>
<link href="/gone.css" media="screen" rel="stylesheet" type="text/css">
<!--[if lte IE 8]><link href="/ie.css" media="screen" rel="stylesheet" type="text/css"><![endif]-->
<% if ENV['RACK_ENV'] != 'development' %>
<script src="/google-analytics.js" type="text/javascript"></script>
<% end %>
</head>
<body>
<div id="wrapper">
Expand Down

0 comments on commit e3ad5e8

Please sign in to comment.