Merge pull request #2283 from bcgov/chore/remove-snowplow

chore: remove snowplow analytics
bcgov · Jun 13, 2024 · bc6e154 · bc6e154
2 parents 9b35a02 + dda0afa
commit bc6e154
Show file tree

Hide file tree

Showing 10 changed files with 318 additions and 1,009 deletions.
diff --git a/app/components/Layout/Analytics.jsx b/app/components/Layout/Analytics.jsx
diff --git a/app/next.config.js b/app/next.config.js
@@ -24,7 +24,6 @@ module.exports = withSentryConfig(
     publicRuntimeConfig: {
       ADMIN_EMAIL: process.env.ADMIN_EMAIL,
       SUPPORT_EMAIL: process.env.SUPPORT_EMAIL,
-      ENABLE_ANALYTICS: process.env.ENABLE_ANALYTICS,
       SITEWIDE_NOTICE: process.env.SITEWIDE_NOTICE,
       ENABLE_DB_MOCKS: process.env.ENABLE_DB_MOCKS,
       SENTRY_ENVIRONMENT: process.env.SENTRY_ENVIRONMENT,

diff --git a/app/package.json b/app/package.json
@@ -135,7 +135,7 @@
     "improved-yarn-audit": "^3.0.0",
     "jest": "^27.2.4",
     "jest-canvas-mock": "^2.3.1",
-    "jest-extended": "^0.11.5",
+    "jest-extended": "^4.0.2",
     "jest-mock-random": "^1.1.1",
     "jest-raw-loader": "^1.0.1",
     "json-schema-faker": "^0.5.0-rc23",
@@ -179,7 +179,7 @@
     "setupFilesAfterEnv": [
       "<rootDir>tests/setupTests.ts",
       "jest-canvas-mock",
-      "jest-extended"
+      "jest-extended/all"
     ],
     "transform": {
       ".*(ts|tsx|js|jsx)$": "babel-jest",

diff --git a/app/pages/_document.tsx b/app/pages/_document.tsx
@@ -1,9 +1,5 @@
 import React from "react";
 import Document, { Html, Head, Main, NextScript } from "next/document";
-import Analytics from "components/Layout/Analytics";
-import getConfig from "next/config";
-
-const CONFIG = getConfig()?.publicRuntimeConfig;
 
 class MyDocument extends Document {
   render() {
@@ -42,7 +38,6 @@ class MyDocument extends Document {
         <body>
           <Main />
           <NextScript />
-          {CONFIG?.ENABLE_ANALYTICS === "true" && <Analytics />}
         </body>
       </Html>
     );

diff --git a/app/server/index.js b/app/server/index.js
@@ -14,7 +14,6 @@ const dev = process.env.NODE_ENV !== "production";
 const app = nextjs({ dev });
 const handle = app.getRequestHandler();
 const bodyParser = require("body-parser");
-const cors = require("cors");
 const UNSUPPORTED_BROWSERS = require("../data/unsupported-browsers");
 const { run } = require("graphile-worker");
 const path = require("path");
@@ -88,9 +87,6 @@ app.prepare().then(async () => {
 
   server.use(bodyParser.json({ limit: "50mb" }));
 
-  // Only allow CORS for the <Analytics /> component
-  server.use(cors({ origin: "https://www2.gov.bc.ca" }));
-
   // Tell search + crawlers not to index non-production environments:
   server.use(({ res, next }) => {
     if (!process.env.NAMESPACE || !process.env.NAMESPACE.endsWith("-prod")) {