Skip to content

Commit ae40fce

Browse files
mwvoloCopilot
andauthored
remove measurements (#1759)
* remove measurements * Remove ExperimentsGuideTest to match removal of experiments_guide feature --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
1 parent e2876e5 commit ae40fce

4 files changed

Lines changed: 0 additions & 67 deletions

File tree

global_settings/templates/experiments_guide.html

Lines changed: 0 additions & 24 deletions
This file was deleted.

global_settings/tests.py

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
11
import re
22

33
from django.test import TestCase, Client
4-
from django.urls import reverse
5-
64
from wagtail.contrib.sitemaps.sitemap_generator import Sitemap
7-
from wagtail.test.utils import WagtailTestUtils
85

96
from global_settings.views import SlashlessSitemap
107

@@ -61,22 +58,6 @@ def test_sitemap_locs_are_slashless(self):
6158
)
6259

6360

64-
class ExperimentsGuideTest(WagtailTestUtils, TestCase):
65-
def setUp(self):
66-
self.login() # creates and logs in a superuser
67-
68-
def test_guide_page_renders_with_links(self):
69-
response = self.client.get(reverse('experiments_guide'))
70-
self.assertEqual(response.status_code, 200)
71-
self.assertContains(response, 'Experiments &amp; Measurement')
72-
self.assertContains(response, 'us.posthog.com')
73-
74-
def test_guide_page_requires_admin_access(self):
75-
self.client.logout()
76-
response = self.client.get(reverse('experiments_guide'))
77-
# require_admin_access redirects anonymous users to the admin login.
78-
self.assertIn(response.status_code, (302, 403))
79-
8061

8162
class WagtailTransferChooserCssHookTest(TestCase):
8263
"""The wagtail-transfer chooser renders its pagination arrows as

global_settings/views.py

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,9 @@
11
from django.contrib.sitemaps import views as sitemap_views
22
from django.http import HttpResponseServerError, HttpResponse
3-
from django.shortcuts import render
4-
from wagtail.admin.auth import require_admin_access
53
from wagtail.contrib.sitemaps.sitemap_generator import Sitemap
64
from global_settings.functions import invalidate_cloudfront_caches
75

86

9-
@require_admin_access
10-
def experiments_guide(request):
11-
return render(request, 'experiments_guide.html', {
12-
'posthog_project_url': 'https://us.posthog.com/project/105101',
13-
'framework_doc': 'https://github.com/openstax/openstax-cms/blob/main/docs/posthog-experiments-measurement.md',
14-
'runbook_doc': 'https://github.com/openstax/openstax-cms/blob/main/docs/posthog-experiments-runbook.md',
15-
})
16-
17-
187
def throw_error(request):
198
# Return an "Internal Server Error" 500 response code.
209
return HttpResponseServerError()

global_settings/wagtail_hooks.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -105,19 +105,6 @@ def register_clear_cache_menu_item():
105105
return MenuItem('Clear Cloudfront Cache', reverse('clear_entire_cache'), classname='icon icon-bin', order=11000)
106106

107107

108-
@hooks.register('register_admin_urls')
109-
def register_experiments_guide_url():
110-
return [path('experiments/', views.experiments_guide, name='experiments_guide')]
111-
112-
113-
@hooks.register('register_settings_menu_item')
114-
def register_experiments_menu_item():
115-
return MenuItem(
116-
'Experiments & Measurement',
117-
reverse('experiments_guide'),
118-
icon_name='bulb',
119-
order=9000,
120-
)
121108
# --- Wagtail Transfer "Import" menu ----------------------------------------
122109
# The package only shows its Import menu item once WAGTAILTRANSFER_SOURCES is
123110
# configured, so it is invisible on a fresh/local environment with no sources.

0 commit comments

Comments
 (0)