Merge pull request #849 from govuk-one-login/OLH-4087-error

OLH-4087: Update authorize error page

Author: danacotoran

solutions/frontend/src/journeys/utils/onRequest.test.ts

@@ -42,7 +42,7 @@ vi.mock(import("../../utils/paths.js"), () => ({
       },
     },
     others: {
-      authorizeError: { path: "/authorize-error" },
+      authorizeError: { path: "/error" },
     },
   },
 }));
@@ -188,7 +188,7 @@ describe("onRequest", () => {
         "Count",
         1,
       );
-      expect(mockReply.redirect).toHaveBeenCalledWith("/authorize-error");
+      expect(mockReply.redirect).toHaveBeenCalledWith("/error");
     });
   });
 
@@ -216,7 +216,7 @@ describe("onRequest", () => {
         "Count",
         1,
       );
-      expect(mockReply.redirect).toHaveBeenCalledWith("/authorize-error");
+      expect(mockReply.redirect).toHaveBeenCalledWith("/error");
     });
   });
 
@@ -250,7 +250,7 @@ describe("onRequest", () => {
         "Count",
         1,
       );
-      expect(mockReply.redirect).toHaveBeenCalledWith("/authorize-error");
+      expect(mockReply.redirect).toHaveBeenCalledWith("/error");
     });
   });
 
@@ -305,7 +305,7 @@ describe("onRequest", () => {
       // Test with a path that has no analytics in the mock
       mockReply.globals = {
         currentUrl: {
-          pathname: "/authorize-error",
+          pathname: "/error",
         } as URL,
       };
 

solutions/frontend/src/templates/layout/base.njk

@@ -27,7 +27,6 @@
   {% endif -%}
   {{ 'general.serviceNameTitle' | translate if not hideTitleProductName}}
 {%- endblock %}
-
 {% block bodyStart %}
   {% include 'templates/layout/cookieBanner.njk' %}
 {% endblock %}

solutions/frontend/src/translations/cy.json

@@ -35,8 +35,8 @@
       "buttonText": "Ewch i hafan GOV.UK"
     },
     "error500": {
-      "title": "Mae’n ddrwg gennym, mae problem gyda’r gwasanaeth",
-      "header": "Mae’n ddrwg gennym, mae problem gyda’r gwasanaeth",
+      "title": "Mae'n ddrwg gennym, mae problem",
+      "header": "Mae'n ddrwg gennym, mae problem",
       "content": {
         "paragraph1": "Rhowch gynnig arall yn nes ymlaen."
       }

solutions/frontend/src/translations/en.json

@@ -35,8 +35,8 @@
       "buttonText": "Go to the GOV.UK homepage"
     },
     "error500": {
-      "title": "Sorry, there is a problem with the service",
-      "header": "Sorry, there is a problem with the service",
+      "title": "Sorry, there is a problem",
+      "header": "Sorry, there is a problem",
       "content": {
         "paragraph1": "Try again later."
       }

solutions/frontend/src/utils/paths.ts

@@ -101,7 +101,13 @@ export const paths = {
   },
   others: {
     authorize: { path: "/authorize" },
-    authorizeError: { path: "/authorize-error" },
+    authorizeError: {
+      path: "/error",
+      analytics: {
+        ...analyticsDefaults,
+        contentId: "a1a3dddd-9e65-40dc-9256-12ed597ec40e",
+      },
+    },
   },
 } as const satisfies {
   journeys: {

solutions/integration-tests/snapshots/Desktop/tests/features/@authorize-error-page.feature.spec.js/Authorize-error-page-Visit-the-authorize-error-page-1.jpg

@@ -14,7 +14,7 @@ const stubsUrl = getStubsUrl();
 export const pageNameToPath: Record<string, string> = {
   "Non-existent page": "/non-existent-page",
   Healthcheck: "/healthcheck",
-  "Authorize error": "/authorize-error",
+  "Authorize error": "/error",
   "Testing journey - step 1": "/testing-journey/step-1",
   "Testing journey - enter password": "/testing-journey/enter-password",
   "Testing journey - confirmation": "/testing-journey/confirm",