Merge pull request #8862 from cfpb/ans_remove_optimizely

Remove google optimize legacy code cruft and AB_TESTING flag

Author: anselmbradford

cfgov/cfgov/settings/base.py

@@ -501,8 +501,6 @@
     "*.googleanalytics.com",
     "*.google-analytics.com",
     "*.googletagmanager.com",
-    "*.googleoptimize.com",
-    "optimize.google.com",
     "api.mapbox.com",
     "js-agent.newrelic.com",
     "bam.nr-data.net",
@@ -524,7 +522,6 @@
     "'unsafe-inline'",
     "*.consumerfinance.gov",
     "*.googletagmanager.com",
-    "optimize.google.com",
     "fonts.googleapis.com",
     "api.mapbox.com",
     "www.ssa.gov/accessibility/andi/",
@@ -539,7 +536,6 @@
     "img.youtube.com",
     "*.google-analytics.com",
     "*.googletagmanager.com",
-    "optimize.google.com",
     "api.mapbox.com",
     "*.tiles.mapbox.com",
     "blob:",
@@ -557,8 +553,6 @@
     "*.consumerfinance.gov",
     "*.googletagmanager.com",
     "*.google-analytics.com",
-    "*.googleoptimize.com",
-    "optimize.google.com",
     "www.youtube.com",
     "*.qualtrics.com",
     "mailto:",
@@ -572,7 +566,6 @@
     "'self'",
     "*.consumerfinance.gov",
     "*.google-analytics.com",
-    "*.googleoptimize.com",
     "*.tiles.mapbox.com",
     "api.mapbox.com",
     "bam.nr-data.net",
@@ -610,10 +603,6 @@
     "CFPB_RECRUITING": [],
     # When enabled, display a "technical issues" banner on /complaintdatabase
     "CCDB_TECHNICAL_ISSUES": [],
-    # Google Optimize code snippets for A/B testing
-    # When enabled this flag will add various Google Optimize code snippets.
-    # Intended for use with path conditions.
-    "AB_TESTING": [],
     # Manually enabled when Beta is being used for an external test.
     # Controls the /beta_external_testing endpoint, which Jenkins jobs
     # query to determine whether to refresh Beta database.

cfgov/v1/jinja2/v1/layouts/base.html

@@ -182,22 +182,6 @@
     {# Don't load certain scripts in the Wagtail preview panel. -#}
     {% if not request.in_preview_panel %}
 
-    {% if flag_enabled('AB_TESTING') %}
-    {# Begin Google Optimize #}
-    {# Optimize anti-flicker snippet. #}
-    <style>.async-hide { opacity: 0 !important} </style>
-    <script>(function(a,s,y,n,c,h,i,d,e){s.className+=' '+y;h.start=1*new Date;
-    h.end=i=function(){s.className=s.className.replace(RegExp(' ?'+y),'')};
-    (a[n]=a[n]||[]).hide=h;setTimeout(function(){i();h.end=null},c);h.timeout=c;
-    })(window,document.documentElement,'async-hide','dataLayer',4000,
-    {'GTM-KMMLRS':true});</script>
-    {#
-      NOTE: We don't include the googleoptimize.com/optimize.js script call here
-      because we are including optimize via GTM.
-    #}
-    {# End Google Optimize #}
-    {% endif %}
-
     {# Begin Google Tag Manager #}
     <script>(function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
     new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],

docs/feature-flags.md

@@ -30,7 +30,7 @@ FLAGS = {
 }
 ```
 
-By convention our flag names are all uppercase, with underscores instead of whitespace. A comment is expected above each flag with a short description fo what happens when it is enabled.
+By convention our flag names are all uppercase, with underscores instead of whitespace. A comment is expected above each flag with a short description of what happens when it is enabled.
 
 ## Checking a flag