From f9574f2168121f0eebb03fa3d55689afc76adcea Mon Sep 17 00:00:00 2001 From: Ben Hodgkiss Date: Mon, 29 Jun 2026 09:28:25 +0100 Subject: [PATCH 1/4] Add LPA Dashboard link to CheckAnswersController --- src/controllers/CheckAnswersController.js | 1 + 1 file changed, 1 insertion(+) diff --git a/src/controllers/CheckAnswersController.js b/src/controllers/CheckAnswersController.js index 38239a80..f4806daa 100644 --- a/src/controllers/CheckAnswersController.js +++ b/src/controllers/CheckAnswersController.js @@ -110,6 +110,7 @@ class CheckAnswersController extends PageController { - Email: ${data.email}\n - Organisation ID: ${data.organisationId}\n - Organisation Name: ${data.organisationName}\n + - LPA Dashboard: https://provide.planning.data.gov.uk/organisations/${data.organisationId}\n - Dataset: ${data.dataset}\n - Documentation URL: ${data.documentationUrl}\n - Endpoint URL: ${data.endpoint}\n From 25dd46dd105063b4fefb15ab6353855fd7971bee Mon Sep 17 00:00:00 2001 From: Ben Hodgkiss Date: Mon, 29 Jun 2026 09:36:41 +0100 Subject: [PATCH 2/4] Update test to check for LPA Dashboard URL Added expectation for LPA Dashboard URL in the test case. --- test/unit/checkAnswersController.test.js | 1 + 1 file changed, 1 insertion(+) diff --git a/test/unit/checkAnswersController.test.js b/test/unit/checkAnswersController.test.js index e2964db6..d8c623b7 100644 --- a/test/unit/checkAnswersController.test.js +++ b/test/unit/checkAnswersController.test.js @@ -125,6 +125,7 @@ describe('CheckAnswersController', () => { const [jiraRequest, jiraRequestTypeId] = createCustomerRequest.mock.calls[0] expect(jiraRequest.description).toContain(`${config.url}check/results/existing-request-id/1`) expect(jiraRequest.description).not.toContain(`${config.url}check/results/existing-request-id/${config.jira.requestTypeId}`) + expect(jiraRequest.description).toContain(`LPA Dashboard: https://provide.planning.data.gov.uk/organisations/${sessionData.orgId}`) expect(jiraRequestTypeId).toBe(config.jira.requestTypeId) expect(createCustomerRequest).toHaveBeenCalledWith( expect.objectContaining({ From 072f09d77ba05351e8365b66eff16b481391bdff Mon Sep 17 00:00:00 2001 From: Ben Hodgkiss Date: Mon, 29 Jun 2026 10:03:32 +0100 Subject: [PATCH 3/4] Update CheckAnswersController.js --- src/controllers/CheckAnswersController.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/controllers/CheckAnswersController.js b/src/controllers/CheckAnswersController.js index f4806daa..7bd5b623 100644 --- a/src/controllers/CheckAnswersController.js +++ b/src/controllers/CheckAnswersController.js @@ -110,7 +110,7 @@ class CheckAnswersController extends PageController { - Email: ${data.email}\n - Organisation ID: ${data.organisationId}\n - Organisation Name: ${data.organisationName}\n - - LPA Dashboard: https://provide.planning.data.gov.uk/organisations/${data.organisationId}\n + - LPA Dashboard: - LPA Dashboard: ${config.url}organisations/${data.organisationId}\n - Dataset: ${data.dataset}\n - Documentation URL: ${data.documentationUrl}\n - Endpoint URL: ${data.endpoint}\n From ba2f93ab0cad0d9510feefe796516a4e12ff7894 Mon Sep 17 00:00:00 2001 From: Ben Hodgkiss Date: Mon, 29 Jun 2026 10:04:00 +0100 Subject: [PATCH 4/4] Update LPA Dashboard URL in test case --- test/unit/checkAnswersController.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/unit/checkAnswersController.test.js b/test/unit/checkAnswersController.test.js index d8c623b7..2223dcb2 100644 --- a/test/unit/checkAnswersController.test.js +++ b/test/unit/checkAnswersController.test.js @@ -125,7 +125,7 @@ describe('CheckAnswersController', () => { const [jiraRequest, jiraRequestTypeId] = createCustomerRequest.mock.calls[0] expect(jiraRequest.description).toContain(`${config.url}check/results/existing-request-id/1`) expect(jiraRequest.description).not.toContain(`${config.url}check/results/existing-request-id/${config.jira.requestTypeId}`) - expect(jiraRequest.description).toContain(`LPA Dashboard: https://provide.planning.data.gov.uk/organisations/${sessionData.orgId}`) + expect(jiraRequest.description).toContain(`LPA Dashboard: ${config.url}organisations/${sessionData.orgId}`) expect(jiraRequestTypeId).toBe(config.jira.requestTypeId) expect(createCustomerRequest).toHaveBeenCalledWith( expect.objectContaining({