Changing the back link to user proper urls rather than just relying o…

…n the browser history as the CSP breaks this

src/controllers/deleteController.ts

@@ -10,7 +10,7 @@ export const showDeleteKey = async (req: Request, res: Response, next: NextFunct
     const apiKey = await ApiService.getKey(user, customerKeyId)
     const createdAt = DashboardPresenter.formatDate(apiKey.CreatedAt, true)
 
-    res.render('delete', { apiKey, createdAt })
+    res.render('delete', { apiKey, createdAt, backLinkHref: '/dashboard' })
   } catch (error) {
     next(error)
   }

src/controllers/keyController.ts

@@ -5,7 +5,7 @@ import { ApiKeyPresenter } from '../presenters/apiKeyPresenter'
 import { validationResult } from 'express-validator'
 
 export const newKey = (_req: Request, res: Response): void => {
-  res.render('newKey')
+  res.render('newKey', { backLinkHref: '/dashboard' })
 }
 
 export const create = async (req: Request, res: Response, next: NextFunction): Promise<void> => {

src/controllers/revokeController.ts

@@ -10,7 +10,7 @@ export const showRevoke = async (req: Request, res: Response, next: NextFunction
     const apiKey = await ApiService.getKey(user, customerKeyId)
     const createdAt = DashboardPresenter.formatDate(apiKey.CreatedAt, true)
 
-    res.render('revoke', { apiKey, createdAt })
+    res.render('revoke', { apiKey, createdAt, backLinkHref: '/dashboard' })
   } catch (error) {
     next(error)
   }

src/controllers/verificationController.ts

@@ -47,7 +47,7 @@ export const newVerificationPage = (req: Request, res: Response): void => {
   session.eoriNumber = session.eoriNumber ?? ''
   session.ukacsReference = session.ukacsReference ?? ''
   session.emailAddress = session.emailAddress ?? ''
-  res.render('verification', { session })
+  res.render('verification', { session, backLinkHref: '/' })
 }
 
 export const checkVerificationDetails = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
@@ -76,9 +76,9 @@ export const checkVerificationDetails = async (req: Request, res: Response, next
       }
     }
     if (Object.keys(errors).length === 0) {
-      res.render('checkVerification', { body, session })
+      res.render('checkVerification', { body, session, backLinkHref: '/verification' })
     } else {
-      res.render('verification', { body, session, errors, errorList: Object.values(errors) })
+      res.render('verification', { body, session, errors, errorList: Object.values(errors), backLinkHref: '/' })
     }
   } catch (error) {
     next(error)
@@ -127,7 +127,7 @@ export const applicationComplete = async (req: Request, res: Response, next: Nex
       req.session = null
       res.render('completion', { applicationReference })
     } else {
-      res.render('checkVerification', { body, session, errors, errorList: Object.values(errors) })
+      res.render('checkVerification', { body, session, errors, errorList: Object.values(errors), backLinkHref: '/verification' })
     }
   } catch (error) {
     next(error)

views/checkVerification.njk

@@ -4,7 +4,6 @@
 {% from "govuk/components/error-summary/macro.njk" import govukErrorSummary %}
 
 {% block content %}
-    {% include "back-button.njk" %}
     <div class="govuk-grid-row">
         <div class="govuk-grid-column-two-thirds-from-desktop">
             <h1 class="govuk-heading-l">Check your answers</h1>

views/completion.njk

@@ -1,6 +1,5 @@
 {% extends "layout.njk" %}
 {% block content %}
-    {% include "back-button.njk" %}
     <main class="govuk-main-wrapper govuk-width-container"
           id="main-content"
           role="main">

views/dashboard.njk

@@ -3,7 +3,6 @@
 {% block title %}Dashboard{% endblock %}
 {% set createKeyPath = ["/dashboard/new"] | join %}
 {% block content %}
-    {% include "back-button.njk" %}
     <span class="govuk-caption-l">Commodity Code Identification Tool Hub</span>
     <h1 class="govuk-heading-l">Your account</h1>
     <p class="govuk-body">

views/delete.njk

@@ -2,7 +2,6 @@
 {% from "govuk/components/warning-text/macro.njk" import govukWarningText %}
 {% block title %}Dashboard{% endblock %}
 {% block content %}
-    {% include "back-button.njk" %}
     <h1 class="govuk-heading-l">Delete an API Key</h1>
     {{ govukWarningText({
         "text": "Your API Key will be deleted with immediate effect and cannot be reactivated once deleted.",

views/keySuccessPage.njk

@@ -8,7 +8,6 @@
 
 {% block title %}Dashboard{% endblock %}
 {% block content %}
-  {% include "back-button.njk" %}
   {{ govukPanel({
       titleText: "Your API Key has been created successfully"
   }) }}

views/layout.njk

@@ -1,5 +1,6 @@
 {% extends "govuk/template.njk" %}
 <!DOCTYPE html>
+{% from "govuk/components/back-link/macro.njk" import govukBackLink %}
 {% from "govuk/components/cookie-banner/macro.njk" import govukCookieBanner %}
 {% from "govuk/components/skip-link/macro.njk" import govukSkipLink %}
 {% from "govuk/components/header/macro.njk" import govukHeader %}
@@ -34,6 +35,12 @@
                 },
                 html: phaseBannerHtml
                 }) }}
+        {%- if backLinkHref %}
+        {{ govukBackLink({
+            href: backLinkHref,
+            text: "Back"
+        }) }}
+        {% endif %}
     {% endblock %}
     <body class="govuk-template_body govuk-frontend-supported">
         <main id="main-content"

views/newKey.njk

@@ -5,7 +5,6 @@
 
 {% block title %}Dashboard{% endblock %}
 {% block content %}
-    {% include "back-button.njk" %}
     {% if error %}
         {{ govukErrorSummary({
             titleText: "There is a problem",

views/rejectedPage.njk

@@ -2,7 +2,6 @@
 {% from "govuk/components/button/macro.njk" import govukButton %}
 
 {% block content %}
-    {% include "back-button.njk" %}
     <div class="govuk-grid-row">
         <div class="govuk-grid-column-two-thirds-from-desktop">
             <h1 class="govuk-heading-l">Your application has been rejected</h1>

views/revoke.njk

@@ -4,7 +4,6 @@
 {% from "govuk/components/warning-text/macro.njk" import govukWarningText %}
 {% block title %}Dashboard{% endblock %}
 {% block content %}
-    {% include "back-button.njk" %}
     <h1 class="govuk-heading-l">Revoke an API Key</h1>
     <p class="govuk-body">
         Check the below details regarding the API Key you have selected to revoke and confirm your choice.

views/verification.njk

@@ -3,7 +3,6 @@
 {% from "govuk/components/error-summary/macro.njk" import govukErrorSummary %}
 
 {% block content %}
-    {% include "back-button.njk" %}
     {% if errorList.length > 0 %}
         {{ govukErrorSummary({
             titleText: "There is a problem",