feat: delete legacy marketing static templates and their URL routes

Remove the 14 template files that were served locally when
ENABLE_MKTG_SITE=False. These pages (about, blog, contact, donate, faq,
help, honor, jobs, media-kit, news, press, privacy, tos, sitemap.xml)
should now only live on the external marketing site configured via
MKTG_URLS.

Also remove the stanford-style theme overrides for about.html and
tos.html, the hardcoded URL patterns for the deleted templates, the
honor.html iframing special case from the render view, and the tests
that exercised these now-deleted routes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: feanil

common/djangoapps/student/tests/test_views.py

@@ -1086,13 +1086,13 @@ def test_user_with_unacknowledged_notice(self, mock_notices):
         Verifies that we will redirect the learner to the URL returned from the `check_for_unacknowledged_notices`
         function.
         """
-        mock_notices.return_value = reverse("about")
+        mock_notices.return_value = reverse("root")
 
         with override_settings(FEATURES={**settings.FEATURES, 'ENABLE_NOTICES': True}):
             response = self.client.get(self.path)
 
         assert response.status_code == 302
-        assert response.url == "/about"
+        assert response.url == "/"
         mock_notices.assert_called_once()
 
     @patch('common.djangoapps.student.views.dashboard.check_for_unacknowledged_notices')

lms/djangoapps/learner_home/test_views.py

@@ -76,12 +76,14 @@ def test_happy_path(self, mock_marketing_link):
 
     @ddt.data(
         (True, f'{settings.CATALOG_MICROFRONTEND_URL}/courses'),
-        (False, '/courses'),
+        (False, '#'),
     )
     @ddt.unpack
     def test_link_with_new_catalog_page(self, catalog_mfe_enabled, expected_catalog_link):
         """
         Test that the catalog link is constructed correctly based on the MFE flags.
+        When the catalog MFE is disabled, marketing_link('COURSES') is used; '#' is
+        returned when MKTG_URLS has no COURSES entry.
         """
         with override_settings(ENABLE_CATALOG_MICROFRONTEND=catalog_mfe_enabled):
             assert get_platform_settings()["courseSearchUrl"] == expected_catalog_link

lms/djangoapps/static_template_view/tests/test_views.py

@@ -7,62 +7,10 @@
 from django.test import TestCase
 from django.urls import reverse
 
-from openedx.core.djangoapps.site_configuration.tests.test_util import with_site_configuration_context
-
 
 class MarketingSiteViewTests(TestCase):
     """ Tests for the marketing site views """
 
-    def _test_view(self, view_name, mimetype):
-        """
-        Gets a view and tests that it exists.
-        """
-        resp = self.client.get(reverse(view_name))
-        assert resp.status_code == 200
-        assert resp['Content-Type'] == mimetype
-
-    def test_sitemap(self):
-        """
-        Test the sitemap view
-        """
-        self._test_view('sitemap_xml', 'application/xml')
-
-    def test_about(self):
-        """
-        Test the about view
-        """
-        self._test_view('about', 'text/html')
-
-    def test_about_with_site_configuration(self):
-        """
-        Test the about view with the header and content set in SiteConfiguration.
-        """
-        test_header = "Very Unique Test Header"
-        test_content = "Very Unique Test Content"
-        test_header_key = 'static_template_about_header'
-        test_content_key = 'static_template_about_content'
-        response = None
-        configuration = {test_header_key: test_header, test_content_key: test_content}
-        with with_site_configuration_context(configuration=configuration):
-            response = self.client.get(reverse("about"))
-        self.assertContains(response, test_header)
-        self.assertContains(response, test_content)
-
-    def test_about_with_site_configuration_and_html(self):
-        """
-        Test the about view with html in the header.
-        """
-        test_header = "<i>Very Unique Test Header</i>"
-        test_content = "<i>Very Unique Test Content</i>"
-        test_header_key = 'static_template_about_header'
-        test_content_key = 'static_template_about_content'
-        response = None
-        configuration = {test_header_key: test_header, test_content_key: test_content}
-        with with_site_configuration_context(configuration=configuration):
-            response = self.client.get(reverse("about"))
-        self.assertContains(response, test_header)
-        self.assertContains(response, test_content)
-
     def test_404(self):
         """
         Test the 404 view.

lms/djangoapps/static_template_view/urls.py

@@ -8,15 +8,6 @@
 from lms.djangoapps.static_template_view import views
 
 urlpatterns = [
-    path('blog', views.render, {'template': 'blog.html'}, name="blog"),
-    path('contact', views.render, {'template': 'contact.html'}, name="contact"),
-    path('donate', views.render, {'template': 'donate.html'}, name="donate"),
-    path('faq', views.render, {'template': 'faq.html'}, name="faq"),
-    path('help', views.render, {'template': 'help.html'}, name="help_edx"),
-    path('jobs', views.render, {'template': 'jobs.html'}, name="jobs"),
-    path('news', views.render, {'template': 'news.html'}, name="news"),
-    path('press', views.render, {'template': 'press.html'}, name="press"),
-    path('media-kit', views.render, {'template': 'media-kit.html'}, name="media-kit"),
     path('copyright', views.render, {'template': 'copyright.html'}, name="copyright"),
 
     # Press releases

lms/djangoapps/static_template_view/views.py

@@ -58,9 +58,6 @@ def render(request, template):
 
     try:
         context = {}
-        # This is necessary for the dialog presented with the TOS in /register
-        if template == 'honor.html':
-            context['allow_iframing'] = True
         # Format Examples: static_template_about_header
         configuration_base = 'static_template_' + template.replace('.html', '').replace('-', '_')
         page_header = configuration_helpers.get_value(configuration_base + '_header')

lms/djangoapps/support/views/contact_us.py

@@ -8,7 +8,7 @@
 from django.shortcuts import redirect
 from django.views.generic import View
 
-from common.djangoapps.edxmako.shortcuts import render_to_response
+from common.djangoapps.edxmako.shortcuts import marketing_link, render_to_response
 from common.djangoapps.student.models import CourseEnrollment
 from openedx.core.djangoapps.site_configuration import helpers as configuration_helpers
 from openedx.features.enterprise_support import api as enterprise_api
@@ -23,7 +23,8 @@ def get(self, request):  # pylint: disable=missing-function-docstring
         # If ZENDESK_URL is not defined, then it will redirect to the static contact page
         # to avoid 500 error that arises due to missing Zendesk configuration
         if not settings.ZENDESK_URL:
-            return redirect('contact')
+            contact_url = marketing_link('CONTACT')
+            return redirect(contact_url if contact_url != '#' else '/')
 
         if not configuration_helpers.get_value('CONTACT_US_PAGE', True):
             raise Http404

lms/templates/signup_modal.html

@@ -4,7 +4,7 @@
 from openedx.core.djangolib.markup import HTML, Text
 from openedx.core.djangolib.js_utils import js_escaped_string
 from django.conf import settings
-from django.urls import reverse
+from common.djangoapps.edxmako.shortcuts import marketing_link
 from django_countries import countries
 from common.djangoapps.student.models import UserProfile
 from datetime import date
@@ -134,15 +134,15 @@ <h2>
           <label data-field="terms_of_service" class="terms-of-service" for="signup_tos">
             <input id="signup_tos" name="terms_of_service" type="checkbox" value="true">
             ${Text(_('I agree to the {link_start}Terms of Service{link_end}')).format(
-              link_start=HTML('<a href="{url}" rel="noopener" target="_blank">').format(url=reverse('tos')),
+              link_start=HTML('<a href="{url}" rel="noopener" target="_blank">').format(url=marketing_link('TOS')),
               link_end=HTML('</a>'))} *
           </label>
 
           % if settings.REGISTRATION_EXTRA_FIELDS['honor_code'] != 'hidden':
           <label data-field="honor_code" class="honor-code" for="signup_honor">
             <input id="signup_honor" name="honor_code" type="checkbox" value="true">
             ${Text(_('I agree to the {link_start}Honor Code{link_end}')).format(
-              link_start=HTML('<a href="{url}" rel="noopener" target="_blank">').format(url=reverse('honor')),
+              link_start=HTML('<a href="{url}" rel="noopener" target="_blank">').format(url=marketing_link('HONOR')),
               link_end=HTML('</a>'))} *
           </label>
           % endif