Skip to content

feat(demo-mode): analytics #89720

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Apr 17, 2025
Merged

feat(demo-mode): analytics #89720

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
074b4e4
feat(demo-mode): analytics
obostjancic Apr 16, 2025
d61ee75
blank line
obostjancic Apr 16, 2025
88f0fba
Merge branch 'master' into ogi/feat/demo-mode-template-analytics
obostjancic Apr 16, 2025
70af49b
domain fix
obostjancic Apr 16, 2025
4e18c18
remove check
obostjancic Apr 16, 2025
9de338d
Merge branch 'master' into ogi/feat/demo-mode-template-analytics
obostjancic Apr 17, 2025
bf4a4a4
use org_service
obostjancic Apr 17, 2025
0f5016f
fix test
obostjancic Apr 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 14 additions & 2 deletions src/sentry/demo_mode/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,15 +41,27 @@ def get_demo_org():
if not is_demo_mode_enabled():
return None

org_id = options.get("demo-mode.orgs")[0]
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.objects.get(id=org_id)


def get_demo_user():
if not is_demo_mode_enabled():
return None

user_id = options.get("demo-mode.users")[0]
demo_users = options.get("demo-mode.users")

if demo_users is None or len(demo_users) == 0:
return None

user_id = demo_users[0]

return User.objects.get(id=user_id)


Expand Down
4 changes: 4 additions & 0 deletions src/sentry/templates/sentry/layout.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
{% load sentry_features %}
{% load sentry_helpers %}
{% load sentry_status %}
{% load sentry_demo_mode %}
{% get_sentry_version %}
<!DOCTYPE html>
<html lang="en">
Expand Down Expand Up @@ -70,6 +71,9 @@
{% asset_url 'sentry' 'js/ads.js' as asset_url %}
{% script src=asset_url crossorigin="anonymous" %}{% endscript %}
{% endblock %}

{% init_demo_analytics %}

</head>


Expand Down
22 changes: 22 additions & 0 deletions src/sentry/templatetags/sentry_demo_mode.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
from django import template
from django.utils.safestring import mark_safe

from sentry.demo_mode.utils import get_demo_org

register = template.Library()


@register.simple_tag
def init_demo_analytics():
org = get_demo_org()

if not org:
return ""

domain = f"{org.slug}.sentry.io"

html = f"""
<script defer data-domain="{domain}" src="https://plausible.io/js/script.pageview-props.tagged-events.js"></script>
<script>window.plausible = window.plausible || function() {{ (window.plausible.q = window.plausible.q || []).push(arguments) }}</script>
"""
return mark_safe(html)
31 changes: 0 additions & 31 deletions static/app/utils/demoMode/utils.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import * as Sentry from '@sentry/react';

import {setForceHide} from 'sentry/actionCreators/guides';
import {demoSignupModal} from 'sentry/actionCreators/modal';
import type {Client} from 'sentry/api';
Expand All @@ -25,7 +23,6 @@ export function initDemoMode(api: Client) {
if (!isDemoModeActive()) {
return;
}
initDemoAnalytics();
captureEmail(api);
}

Expand Down Expand Up @@ -57,31 +54,3 @@ 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);
}
}
24 changes: 24 additions & 0 deletions tests/sentry/demo_mode/test_utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,18 @@ 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():
Expand All @@ -101,3 +113,15 @@ def test_get_demo_org_demo_mode_enabled():
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
Loading