Revert "feat(demo-mode): analytics (#89720)"

getsentry-bot · obostjancic · getsentry-bot · commit b0eba8836d43 · 2025-04-17T11:58:15.000Z
This reverts commit fcc4460. Co-authored-by: obostjancic <86684834+obostjancic@users.noreply.github.com>
diff --git a/src/sentry/demo_mode/utils.py b/src/sentry/demo_mode/utils.py
@@ -2,7 +2,6 @@
 
 from sentry import options
 from sentry.models.organization import Organization
-from sentry.organizations.services.organization import organization_service
 from sentry.users.models.user import User
 
 READONLY_SCOPES = frozenset(
@@ -42,29 +41,15 @@ def get_demo_org():
     if not is_demo_mode_enabled():
         return None
 
-    demo_orgs = options.get("demo-mode.orgs")
-
-    if demo_orgs is None or len(demo_orgs) == 0:
-        return None
-
-    org_id = demo_orgs[0]
-
-    return organization_service.get_org_by_id(
-        id=org_id,
-    )
+    org_id = options.get("demo-mode.orgs")[0]
+    return Organization.objects.get(id=org_id)
 
 
 def get_demo_user():
     if not is_demo_mode_enabled():
         return None
 
-    demo_users = options.get("demo-mode.users")
-
-    if demo_users is None or len(demo_users) == 0:
-        return None
-
-    user_id = demo_users[0]
-
+    user_id = options.get("demo-mode.users")[0]
     return User.objects.get(id=user_id)
 
 
diff --git a/src/sentry/templates/sentry/layout.html b/src/sentry/templates/sentry/layout.html
@@ -4,7 +4,6 @@
 {% load sentry_features %}
 {% load sentry_helpers %}
 {% load sentry_status %}
-{% load sentry_demo_mode %}
 {% get_sentry_version %}
 <!DOCTYPE html>
 <html lang="en">
@@ -71,9 +70,6 @@
   {% asset_url 'sentry' 'js/ads.js' as asset_url %}
   {% script src=asset_url crossorigin="anonymous" %}{% endscript %}
   {% endblock %}
-
-  {% init_demo_analytics %}
-
 </head>
 
 
diff --git a/src/sentry/templatetags/sentry_demo_mode.py b/src/sentry/templatetags/sentry_demo_mode.py
diff --git a/static/app/utils/demoMode/utils.tsx b/static/app/utils/demoMode/utils.tsx
@@ -1,3 +1,5 @@
+import * as Sentry from '@sentry/react';
+
 import {setForceHide} from 'sentry/actionCreators/guides';
 import {demoSignupModal} from 'sentry/actionCreators/modal';
 import type {Client} from 'sentry/api';
@@ -23,6 +25,7 @@ export function initDemoMode(api: Client) {
   if (!isDemoModeActive()) {
     return;
   }
+  initDemoAnalytics();
   captureEmail(api);
 }
 
@@ -54,3 +57,31 @@ async function captureEmail(api: Client) {
     setForceHide(false);
   }
 }
+
+function initDemoAnalytics() {
+  if (!isDemoModeActive()) {
+    return;
+  }
+
+  if (document.getElementById('plausible-script')) {
+    return;
+  }
+
+  try {
+    const mainScript = document.createElement('script');
+    mainScript.id = 'plausible-script';
+    mainScript.defer = true;
+    mainScript.setAttribute('data-domain', window.location.hostname);
+    mainScript.src = 'https://plausible.io/js/script.pageview-props.tagged-events.js';
+
+    const queueScript = document.createElement('script');
+    queueScript.id = 'plausible-queue-script';
+    queueScript.textContent =
+      'window.plausible = window.plausible || function() { (window.plausible.q = window.plausible.q || []).push(arguments) }';
+
+    document.head.appendChild(mainScript);
+    document.head.appendChild(queueScript);
+  } catch (error) {
+    Sentry.captureException(error);
+  }
+}
diff --git a/tests/sentry/demo_mode/test_utils.py b/tests/sentry/demo_mode/test_utils.py
@@ -79,18 +79,6 @@ def test_get_demo_user_demo_mode_disabled():
     assert get_demo_user() is None
 
 
-@override_options({"demo-mode.enabled": True, "demo-mode.users": None})
-@django_db_all
-def test_get_demo_user_demo_mode_enabled_users_none():
-    assert get_demo_user() is None
-
-
-@override_options({"demo-mode.enabled": True, "demo-mode.users": []})
-@django_db_all
-def test_get_demo_user_demo_mode_enabled_users_empty():
-    assert get_demo_user() is None
-
-
 @override_options({"demo-mode.enabled": True, "demo-mode.users": [1]})
 @django_db_all
 def test_get_demo_user_demo_mode_enabled():
@@ -110,20 +98,6 @@ def test_get_demo_org_demo_mode_disabled():
 @django_db_all
 def test_get_demo_org_demo_mode_enabled():
     org = Factories.create_organization(id=1)
-    with patch(
-        "sentry.demo_mode.utils.organization_service.get_org_by_id", return_value=org
-    ) as mock_org_get:
+    with patch("sentry.demo_mode.utils.Organization.objects.get", return_value=org) as mock_org_get:
         assert get_demo_org() == org
         mock_org_get.assert_called_once_with(id=1)
-
-
-@override_options({"demo-mode.enabled": True, "demo-mode.orgs": []})
-@django_db_all
-def test_get_demo_org_demo_mode_enabled_not_set():
-    assert get_demo_org() is None
-
-
-@override_options({"demo-mode.enabled": True, "demo-mode.orgs": None})
-@django_db_all
-def test_get_demo_org_demo_mode_enabled_orgs_none():
-    assert get_demo_org() is None
