From 8649d23810f78f4232be6f40b49b92fca9a096c8 Mon Sep 17 00:00:00 2001
From: Jan Thijs <92784122+janthijs@users.noreply.github.com>
Date: Tue, 7 Nov 2023 15:50:51 +0100
Subject: [PATCH 1/2] fix: use name of vergunning for content
---
.../pages/VergunningDetail/RvvSloterweg.tsx | 14 +++++------
src/server/mock-data/json/vergunningen.json | 24 +++++++++----------
.../vergunningen/vergunningen-content.ts | 14 +++++------
3 files changed, 26 insertions(+), 26 deletions(-)
diff --git a/src/client/pages/VergunningDetail/RvvSloterweg.tsx b/src/client/pages/VergunningDetail/RvvSloterweg.tsx
index 5d2e50b002..b0ecb36671 100644
--- a/src/client/pages/VergunningDetail/RvvSloterweg.tsx
+++ b/src/client/pages/VergunningDetail/RvvSloterweg.tsx
@@ -30,6 +30,10 @@ export function getRVVSloterwegLineItems(
RVV_SLOTERWEG_RESULT_MATURED,
].includes(vergunning.decision);
+ const grantedText = isChangeRequest
+ ? `Wij hebben uw aanvraag voor een ${vergunning.title} verleend.`
+ : `Wij hebben uw kentekenwijziging voor een ${vergunning.title} verleend.`;
+
let dateInProgress = vergunning.dateWorkflowActive ?? '';
if (!isChangeRequest) {
@@ -63,10 +67,7 @@ export function getRVVSloterwegLineItems(
: !!vergunning.dateDecision
? vergunning.dateDecision
: '',
- description:
- isGranted && !(hasDecision || isExpired)
- ? 'Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.'
- : '',
+ description: isGranted && !(hasDecision || isExpired) ? grantedText : '',
documents: [],
isActive: isGranted && !(hasDecision || isExpired),
isChecked: isGranted || hasDecision || isExpired,
@@ -79,12 +80,11 @@ export function getRVVSloterwegLineItems(
isExpired ||
vergunning.decision === RVV_SLOTERWEG_RESULT_UPDATED_WIHT_NEW_KENTEKEN
) {
- description = 'Uw RVV ontheffing Sloterweg is verlopen.';
+ description = `Uw ${vergunning.title} is verlopen.`;
}
if (vergunning.decision === RVV_SLOTERWEG_RESULT_NOT_APPLICABLE) {
- description =
- 'Wij hebben uw RVV ontheffing Sloterweg ingetrokken. Bekijk het document voor details.';
+ description = `Wij hebben uw ${vergunning.title} ingetrokken. Zie het intrekkingsbesluit voor meer informatie.`;
}
if (vergunning.decision === RVV_SLOTERWEG_RESULT_MATURED) {
diff --git a/src/server/mock-data/json/vergunningen.json b/src/server/mock-data/json/vergunningen.json
index 4ff0689249..4391228b61 100644
--- a/src/server/mock-data/json/vergunningen.json
+++ b/src/server/mock-data/json/vergunningen.json
@@ -1119,7 +1119,7 @@
{
"caseType": "RVV Sloterweg",
"id": "unique-id-of-zaak",
- "title": "RVV ontheffing Sloterweg (Wijziging/Ontvangen)",
+ "title": "RVV ontheffing van Vlaanderenlaan (Wijziging/Ontvangen)",
"identifier": "Z/23/98989234",
"dateRequest": "2023-07-24T11:21:33",
"status": "Ontvangen",
@@ -1138,7 +1138,7 @@
{
"caseType": "RVV Sloterweg",
"id": "unique-id-of-zaak",
- "title": "RVV ontheffing Sloterweg (Wijziging/Ingetrokken)",
+ "title": "RVV ontheffing van Vlaanderenlaan (Wijziging/Ingetrokken)",
"identifier": "Z/23/23423409",
"dateRequest": "2023-07-24T11:21:33",
"status": "Ontvangen",
@@ -1171,7 +1171,7 @@
"licensePlates": "BB-344-E",
"previousLicensePlates": "ZZ-122-C",
"requestType": "Wijziging",
- "area": "van Vlaanderenlaan"
+ "area": "Sloterweg"
},
{
"caseType": "RVV Sloterweg",
@@ -1190,7 +1190,7 @@
"licensePlates": "BB-344-Q",
"previousLicensePlates": "YY-123-B",
"requestType": "Wijziging",
- "area": "Laan van Vlaanderen"
+ "area": "Sloterweg"
},
{
"caseType": "RVV Sloterweg",
@@ -1209,12 +1209,12 @@
"licensePlates": "ZZ-122-A",
"previousLicensePlates": "",
"requestType": "Nieuw",
- "area": "Laan van Vlaanderen"
+ "area": "Sloterweg"
},
{
"caseType": "RVV Sloterweg",
"id": "unique-id-of-zaak",
- "title": "RVV ontheffing Sloterweg (Nieuw/Ingetrokken)",
+ "title": "RVV ontheffing Laan van Vlaanderen (Nieuw/Ingetrokken)",
"identifier": "Z/23/123123456",
"dateRequest": "2023-07-01T11:21:33",
"status": "iets?",
@@ -1231,7 +1231,7 @@
"area": "Laan van Vlaanderen"
},
{
- "area": "Laan van Vlaanderen",
+ "area": "Sloterweg",
"caseType": "RVV Sloterweg",
"dateDecision": null,
"dateEnd": "2025-08-31",
@@ -1272,14 +1272,14 @@
"processed": true,
"requestType": "Nieuw",
"status": "Afgehandeld",
- "title": "RVV ontheffing Sloterweg",
+ "title": "RVV ontheffing Sloterweg West",
"link": {
"to": "/vergunningen/rvv-sloterweg/394540596",
"title": "Bekijk hoe het met uw aanvraag staat"
}
},
{
- "area": "Laan van Vlaanderen",
+ "area": "Sloterweg West",
"caseType": "RVV Sloterweg",
"dateDecision": null,
"dateEnd": "2025-08-31",
@@ -1296,14 +1296,14 @@
"processed": false,
"requestType": "Wijziging",
"status": "Actief",
- "title": "RVV ontheffing Sloterweg",
+ "title": "RVV ontheffing Sloterweg West",
"link": {
"to": "/vergunningen/rvv-sloterweg/1623553971",
"title": "Bekijk hoe het met uw aanvraag staat"
}
},
{
- "area": "Laan van Vlaanderen",
+ "area": "Sloterweg West",
"caseType": "RVV Sloterweg",
"dateDecision": null,
"dateEnd": "2025-08-19",
@@ -1320,7 +1320,7 @@
"processed": false,
"requestType": "Nieuw",
"status": "Ontvangen",
- "title": "RVV ontheffing Sloterweg",
+ "title": "RVV ontheffing Sloterweg West",
"link": {
"to": "/vergunningen/rvv-sloterweg/4069415615",
"title": "Bekijk hoe het met uw aanvraag staat"
diff --git a/src/server/services/vergunningen/vergunningen-content.ts b/src/server/services/vergunningen/vergunningen-content.ts
index 58a22bb793..df65110745 100644
--- a/src/server/services/vergunningen/vergunningen-content.ts
+++ b/src/server/services/vergunningen/vergunningen-content.ts
@@ -285,7 +285,7 @@ export const notificationContent: NotificationContent = {
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg ${
+ } ${item.title} ${
(item as RVVSloterweg).requestType === 'Wijziging' ? 'naar ' : ''
}(${(item as RVVSloterweg).licensePlates}) ontvangen.`,
},
@@ -296,13 +296,13 @@ export const notificationContent: NotificationContent = {
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg in behandeling`,
+ } ${item.title} in behandeling`,
description: (item) =>
`Wij hebben uw aanvraag voor een${
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg ${
+ } ${item.title} ${
(item as RVVSloterweg).requestType === 'Wijziging' ? 'naar ' : ''
}(${(item as RVVSloterweg).licensePlates}) in behandeling genomen`,
},
@@ -313,13 +313,13 @@ export const notificationContent: NotificationContent = {
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg verleend`,
+ } ${item.title} verleend`,
description: (item) =>
`Wij hebben uw aanvraag voor een${
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg ${
+ } ${item.title} ${
(item as RVVSloterweg).requestType === 'Wijziging' ? 'naar ' : ''
}(${(item as RVVSloterweg).licensePlates}) verleend`,
},
@@ -330,13 +330,13 @@ export const notificationContent: NotificationContent = {
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg ingetrokken`,
+ } ${item.title} ingetrokken`,
description: (item) =>
`Wij hebben uw aanvraag voor een${
(item as RVVSloterweg).requestType === 'Wijziging'
? ' kentekenwijziging'
: ''
- } RVV ontheffing Sloterweg ${
+ } ${item.title} ${
(item as RVVSloterweg).requestType === 'Wijziging' ? 'naar ' : ''
}(${(item as RVVSloterweg).licensePlates}) ingetrokken`,
},
From 02820d0c816109eb75de2d4fd1553fc31d0b4384 Mon Sep 17 00:00:00 2001
From: Jan Thijs <92784122+janthijs@users.noreply.github.com>
Date: Tue, 7 Nov 2023 15:53:15 +0100
Subject: [PATCH 2/2] test: update snapshots
---
.../VergunningDetail.test.tsx.snap | 26 ++++++-------
.../__snapshots__/Vergunningen.test.tsx.snap | 12 +++---
.../__snapshots__/vergunningen.test.ts.snap | 38 +++++++++----------
3 files changed, 38 insertions(+), 38 deletions(-)
diff --git a/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap b/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap
index 5c99394def..4d81dae189 100644
--- a/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap
+++ b/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap
@@ -4580,7 +4580,7 @@ exports[`
- Laan van Vlaanderen + Sloterweg
> RVV Sloterweg > RVV ontheffing Sloterweg (Nieuw/- van Vlaanderenlaan + Sloterweg
> RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig- Laan van Vlaanderen + Sloterweg
> RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig