Skip to content

Commit 4873eba

Browse files
UKVIET-79: Update Feedback Url (#229)
- added ref to env feedback url (config.js) - set value of feedback url to config value (server.js) - updated node version in drone (.drone.yml) - upgraded HOF package and dependencies (yarn.lock) - removed devDependency funkie-chromedriver (package.json)
1 parent 0e0d922 commit 4873eba

File tree

5 files changed

+281
-553
lines changed

5 files changed

+281
-553
lines changed

.drone.yml

+5-5
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,15 @@ trigger:
2525

2626
linting: &linting
2727
pull: if-not-exists
28-
image: node:lts
28+
image: node:20
2929
environment:
3030
NOTIFY_KEY: USE_MOCK
3131
commands:
3232
- yarn run test:lint
3333

3434
unit_tests: &unit_tests
3535
pull: if-not-exists
36-
image: node:lts
36+
image: node:20
3737
environment:
3838
NOTIFY_KEY: USE_MOCK
3939
commands:
@@ -67,7 +67,7 @@ steps:
6767

6868
- name: setup_deploy
6969
pull: if-not-exists
70-
image: node:lts
70+
image: node:20
7171
environment:
7272
NOTIFY_KEY: USE_MOCK
7373
commands:
@@ -176,7 +176,7 @@ steps:
176176

177177
- name: setup_branch
178178
pull: if-not-exists
179-
image: node:lts
179+
image: node:20
180180
environment:
181181
NOTIFY_KEY: USE_MOCK
182182
commands:
@@ -233,7 +233,7 @@ steps:
233233
# Snyk security scans which run after branch deployment to prevent blocking of PR UAT tests
234234
- name: snyk_scan
235235
pull: if-not-exists
236-
image: node:lts
236+
image: node:20
237237
environment:
238238
SNYK_TOKEN:
239239
from_secret: snyk_token

config.js

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ module.exports = {
99
DATE_FORMAT: 'YYYY-MM-DD',
1010
PRETTY_DATE_FORMAT: 'Do MMMM YYYY',
1111
dateTimeFormat: 'DD MMM YYYY HH:MM:SS ZZ',
12+
feedbackUrl: process.env.FEEDBACK_URL,
1213
notify: {
1314
apiKey: process.env.NOTIFY_STUB === 'true' ? 'USE_MOCK' : process.env.NOTIFY_KEY,
1415
caseworkerEmail: process.env.CASEWORKER_EMAIL,

package.json

-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@
5151
"eslint-config-hof": "^1.2.1",
5252
"express": "^4.17.1",
5353
"funkie": "^0.0.6",
54-
"funkie-chromedriver": "^0.1.0",
5554
"mocha": "^9.0.3",
5655
"nyc": "^15.1.0",
5756
"playwright": "^1.39.0",

server.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ const app = hof(settings);
3535
app.use((req, res, next) => {
3636
res.locals.htmlLang = 'en';
3737
// Set feedback url, required to display phase banner
38-
res.locals.feedbackUrl = 'https://eforms.homeoffice.gov.uk/outreach/feedback.ofml';
38+
res.locals.feedbackUrl = config.feedbackUrl;
3939
res.locals.startPageRedirectUrl = config.startPageRedirectUrl;
4040
res.locals.footerSupportLinks = [
4141
{ path: '/cookies', property: 'base.cookies' },

0 commit comments

Comments
 (0)