diff --git a/src/registrar/tests/test_views_domains_json.py b/src/registrar/tests/test_views_domains_json.py index b18ae1261..90b121f08 100644 --- a/src/registrar/tests/test_views_domains_json.py +++ b/src/registrar/tests/test_views_domains_json.py @@ -76,7 +76,7 @@ class GetDomainsJsonTest(TestWithUser, WebTest): self.assertEqual(expected_domain.state_display(), state_displays[i]) self.assertEqual(expected_domain.get_state_help_text(), get_state_help_texts[i]) - self.assertEqual(f"/domain/{expected_domain.id}", action_urls[i]) + self.assertEqual( reverse("domain", kwargs={"pk": expected_domain.id}), action_urls[i]) # Check action_label action_label_expected = ( diff --git a/src/registrar/views/domain_requests_json.py b/src/registrar/views/domain_requests_json.py index 447ccc6d4..c602269c6 100644 --- a/src/registrar/views/domain_requests_json.py +++ b/src/registrar/views/domain_requests_json.py @@ -3,6 +3,7 @@ from django.core.paginator import Paginator from registrar.models import DomainRequest from django.utils.dateformat import format from django.contrib.auth.decorators import login_required +from django.urls import reverse @login_required @@ -33,14 +34,14 @@ def get_domain_requests_json(request): "is_deletable": domain_request.status in [DomainRequest.DomainRequestStatus.STARTED, DomainRequest.DomainRequestStatus.WITHDRAWN], "action_url": ( - f"/domain-request/{domain_request.id}/edit" + reverse("edit-domain-request", kwargs={"pk": domain_requests.id}) if domain_request.status in [ DomainRequest.DomainRequestStatus.STARTED, DomainRequest.DomainRequestStatus.ACTION_NEEDED, DomainRequest.DomainRequestStatus.WITHDRAWN, ] - else f"/domain-request/{domain_request.id}" + else reverse("domain-request-status", kwargs={"pk": domain_request.id}) ), "action_label": ( "Edit"