Skip to content
Snippets Groups Projects
Unverified Commit 38d540a9 authored by Nemanja Glumac's avatar Nemanja Glumac Committed by GitHub
Browse files

Remove code related to GoogleAnalytics tracking (#42556)

parent 77fccd8b
No related branches found
No related tags found
No related merge requests found
......@@ -38,8 +38,6 @@
engines
enum-cardinality-threshold
follow-up-email-sent
ga-code
ga-enabled
google-auth-auto-create-accounts-domain
google-auth-client-id
google-auth-configured
......
......@@ -178,7 +178,6 @@ export const createMockSettings = (
"has-user-setup": true,
"hide-embed-branding?": true,
"show-static-embed-terms": true,
"ga-enabled": false,
"google-auth-auto-create-accounts-domain": null,
"google-auth-client-id": null,
"google-auth-configured": false,
......
......@@ -281,7 +281,6 @@ interface PublicSettings {
"enable-enhancements?": boolean;
"enable-password-login": boolean;
engines: Record<string, Engine>;
"ga-enabled": boolean;
"google-auth-client-id": string | null;
"google-auth-enabled": boolean;
"has-user-setup": boolean;
......
......@@ -5,15 +5,9 @@ import Settings from "metabase/lib/settings";
import { getUserId } from "metabase/selectors/user";
export const createTracker = store => {
if (Settings.googleAnalyticsEnabled()) {
createGoogleAnalyticsTracker();
}
if (Settings.snowplowEnabled()) {
createSnowplowTracker(store);
}
if (Settings.googleAnalyticsEnabled() || Settings.snowplowEnabled()) {
document.body.addEventListener("click", handleStructEventClick, true);
}
};
......@@ -23,10 +17,6 @@ export const trackPageView = url => {
return;
}
if (Settings.googleAnalyticsEnabled()) {
trackGoogleAnalyticsPageView(getSanitizedUrl(url));
}
if (Settings.snowplowEnabled()) {
trackSnowplowPageView(getSanitizedUrl(url));
}
......@@ -39,10 +29,6 @@ export const trackStructEvent = (category, action, label, value) => {
if (!category || !label || !Settings.trackingEnabled()) {
return;
}
if (Settings.googleAnalyticsEnabled()) {
trackGoogleAnalyticsStructEvent(category, action, label, value);
}
};
export const trackSchemaEvent = (schema, version, data) => {
......@@ -66,28 +52,6 @@ export const trackSchemaEvent = (schema, version, data) => {
}
};
const createGoogleAnalyticsTracker = () => {
const code = Settings.get("ga-code");
window.ga?.("create", code, "auto");
Settings.on("anon-tracking-enabled", enabled => {
window[`ga-disable-${code}`] = enabled ? null : true;
});
};
const trackGoogleAnalyticsPageView = url => {
const version = Settings.get("version", {});
window.ga?.("set", "dimension1", version.tag);
window.ga?.("set", "page", url);
window.ga?.("send", "pageview", url);
};
const trackGoogleAnalyticsStructEvent = (category, action, label, value) => {
const version = Settings.get("version", {});
window.ga?.("set", "dimension1", version.tag);
window.ga?.("send", "event", category, action, label, value);
};
const createSnowplowTracker = store => {
Snowplow.newTracker("sp", Settings.snowplowUrl(), {
appId: "metabase",
......
......@@ -220,13 +220,6 @@ class MetabaseSettings {
return !!(this.get("uploads-enabled") && this.get("uploads-database-id"));
}
/**
* @deprecated use getSetting(state, "ga-enabled")
*/
googleAnalyticsEnabled() {
return this.get("ga-enabled") || false;
}
/**
* @deprecated use getSetting(state, "snowplow-enabled")
*/
......
......@@ -214,21 +214,6 @@
:visibility :public
:audit :getter)
(defsetting ga-code
(deferred-tru "Google Analytics tracking code.")
:default "UA-60817802-1"
:visibility :public
:doc false)
(defsetting ga-enabled
(deferred-tru "Boolean indicating whether analytics data should be sent to Google Analytics on the frontend")
:type :boolean
:setter :none
:getter (fn [] (and config/is-prod? (anon-tracking-enabled)))
:visibility :public
:audit :never
:doc false)
(defsetting map-tile-server-url
(deferred-tru "The map tile server URL template used in map visualizations, for example from OpenStreetMaps or MapBox.")
:default "https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment