Rename "add_pd" to "add_pdc" (PagerDuty Connect).

hc/front/tests/test_add_pdc.py

@@ -4,8 +4,8 @@ from hc.test import BaseTestCase
 
 
 @override_settings(PD_VENDOR_KEY="foo")
-class AddPdTestCase(BaseTestCase):
+class AddPdConnectTestCase(BaseTestCase):
-    url = "/integrations/add_pd/"
+    url = "/integrations/add_pdc/"
 
     def test_instructions_work(self):
         self.client.login(username="alice@example.org", password="password")
@@ -18,7 +18,7 @@ class AddPdTestCase(BaseTestCase):
         session.save()
 
         self.client.login(username="alice@example.org", password="password")
-        url = "/integrations/add_pd/1234567890AB/?service_key=123"
+        url = self.url + "1234567890AB/?service_key=123"
         r = self.client.get(url)
         self.assertEqual(r.status_code, 302)
 
@@ -33,11 +33,11 @@ class AddPdTestCase(BaseTestCase):
         session.save()
 
         self.client.login(username="alice@example.org", password="password")
-        url = "/integrations/add_pd/XXXXXXXXXXXX/?service_key=123"
+        url = self.url + "XXXXXXXXXXXX/?service_key=123"
         r = self.client.get(url)
         self.assertEqual(r.status_code, 400)
 
     @override_settings(PD_VENDOR_KEY=None)
     def test_it_requires_vendor_key(self):
-        r = self.client.get("/integrations/add_pd/")
+        r = self.client.get(self.url)
         self.assertEqual(r.status_code, 404)

hc/front/urls.py

@@ -27,8 +27,8 @@ channel_urls = [
     path("add_email/", views.add_email, name="hc-add-email"),
     path("add_webhook/", views.add_webhook, name="hc-add-webhook"),
     path("add_shell/", views.add_shell, name="hc-add-shell"),
-    path("add_pd/", views.add_pd, name="hc-add-pd"),
+    path("add_pdc/", views.add_pdc, name="hc-add-pdc"),
-    path("add_pd/<str:state>/", views.add_pd, name="hc-add-pd-state"),
+    path("add_pdc/<str:state>/", views.add_pdc, name="hc-add-pdc-state"),
     path("add_pagertree/", views.add_pagertree, name="hc-add-pagertree"),
     path("add_pagerteam/", views.add_pagerteam, name="hc-add-pagerteam"),
     path("add_slack/", views.add_slack, name="hc-add-slack"),

hc/front/views.py

@@ -863,7 +863,7 @@ def _get_validated_code(request, session_key, key="code"):
     return request.GET.get(key)
 
 
-def add_pd(request, state=None):
+def add_pdc(request, state=None):
     if settings.PD_VENDOR_KEY is None:
         raise Http404("pagerduty integration is not available")
 
@@ -893,13 +893,13 @@ def add_pd(request, state=None):
         return redirect("hc-channels")
 
     state = _prepare_state(request, "pd")
-    callback = settings.SITE_ROOT + reverse("hc-add-pd-state", args=[state])
+    callback = settings.SITE_ROOT + reverse("hc-add-pdc-state", args=[state])
     connect_url = "https://connect.pagerduty.com/connect?" + urlencode(
         {"vendor": settings.PD_VENDOR_KEY, "callback": callback}
     )
 
     ctx = {"page": "channels", "project": request.project, "connect_url": connect_url}
-    return render(request, "integrations/add_pd.html", ctx)
+    return render(request, "integrations/add_pdc.html", ctx)
 
 
 @login_required

templates/front/channels.html

@@ -259,7 +259,7 @@
             <h2>PagerDuty</h2>
             <p>On-call scheduling, alerting, and incident tracking.</p>
 
-            <a href="{% url 'hc-add-pd' %}" class="btn btn-primary">Add Integration</a>
+            <a href="{% url 'hc-add-pdc' %}" class="btn btn-primary">Add Integration</a>
         </li>
         {% endif %}