[feat-6019]: Update 'when' question for THOR and VTH subcategories

src/signals/incident/definitions/__tests__/wizard-step-4-summary.test.js

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MPL-2.0
-// Copyright (C) 2020 - 2023 Gemeente Amsterdam
+// Copyright (C) 2020 - 2024 Gemeente Amsterdam
 import configuration from 'shared/services/configuration/configuration'
 
 import PreviewComponents from '../../components/IncidentPreview/components'
@@ -149,25 +149,6 @@ describe('Wizard summary', () => {
             optional: true,
             render: PreviewComponents.DateTime,
           },
-          extra_afval_handhaving: {
-            label: 'Welk afval is verkeerd neergezet?',
-            optional: true,
-            render: Label,
-            canBeNull: false,
-          },
-          extra_afval_handhaving_owner: {
-            label:
-              'Weet u wie de eigenaar is van het verkeerd geplaatste afval? Bijvoorbeeld omdat u dat ziet aan een adressticker of iets anders?',
-            optional: true,
-            render: ObjectLabel,
-            canBeNull: false,
-          },
-          extra_afval_handhaving_owner_confirmation: {
-            canBeNull: false,
-            label: undefined,
-            optional: true,
-            render: Null,
-          },
           extra_afval: {
             label: 'Waar komt het afval vandaan, denkt u?',
             optional: true,

src/signals/incident/definitions/wizard-step-2-vulaan.js

@@ -1,11 +1,12 @@
 // SPDX-License-Identifier: MPL-2.0
-// Copyright (C) 2018 - 2023 Gemeente Amsterdam
+// Copyright (C) 2018 - 2024 Gemeente Amsterdam
 import memoize from 'lodash/memoize'
 
 import configuration from 'shared/services/configuration/configuration'
 
 import afval from './wizard-step-2-vulaan/afval'
 import afvalContainer from './wizard-step-2-vulaan/afval-container'
+import afvalThor from './wizard-step-2-vulaan/afval-thor'
 import boomIllegaleKap from './wizard-step-2-vulaan/boom-illegale-kap'
 import bouwSloopOverlast from './wizard-step-2-vulaan/bouw-sloop-overlast'
 import civieleConstructies from './wizard-step-2-vulaan/civieleConstructies'
@@ -15,9 +16,11 @@ import locatie from './wizard-step-2-vulaan/locatie'
 import overlastBedrijvenEnHoreca from './wizard-step-2-vulaan/overlast-bedrijven-en-horeca'
 import overlastInDeOpenbareRuimte from './wizard-step-2-vulaan/overlast-in-de-openbare-ruimte'
 import overlastOpHetWater from './wizard-step-2-vulaan/overlast-op-het-water'
+import overlastOpHetWaterThor from './wizard-step-2-vulaan/overlast-op-het-water-thor'
 import overlastVanDieren from './wizard-step-2-vulaan/overlast-van-dieren'
 import overlastPersonenEnGroepen from './wizard-step-2-vulaan/overlast-van-en-door-personen-of-groepen'
 import straatverlichtingKlokken from './wizard-step-2-vulaan/straatverlichting-klokken'
+import verkeersoverlast from './wizard-step-2-vulaan/verkeersoverlast'
 import wegenVerkeerStraatmeubilair from './wizard-step-2-vulaan/wegen-verkeer-straatmeubilair'
 import wonen from './wizard-step-2-vulaan/wonen'
 import FormComponents from '../components/form'
@@ -107,6 +110,9 @@ export default {
         if (subcategory.startsWith('container')) {
           return expandQuestions(afvalContainer, category, subcategory)
         }
+        if (['asbest-accu', 'handhaving-op-afval'].includes(subcategory)) {
+          return expandQuestions(afvalThor, category, subcategory)
+        }
         return expandQuestions(afval, category, subcategory)
       }
 
@@ -141,6 +147,18 @@ export default {
       }
 
       case 'overlast-op-het-water':
+        if (
+          [
+            'blokkade-van-de-vaarweg',
+            'overig-boten',
+            'overlast-op-het-water-geluid',
+            'overlast-op-het-water-snel-varen',
+            'scheepvaart-nautisch-toezicht',
+          ].includes(subcategory)
+        ) {
+          return expandQuestions(overlastOpHetWaterThor, category, subcategory)
+        }
+
         return expandQuestions(overlastOpHetWater, category, subcategory)
 
       case 'overlast-van-dieren':
@@ -150,13 +168,23 @@ export default {
         return expandQuestions(overlastPersonenEnGroepen, category, subcategory)
 
       case 'wegen-verkeer-straatmeubilair': {
-        const config = ['klok', 'lantaarnpaal-straatverlichting'].includes(
+        if (['klok', 'lantaarnpaal-straatverlichting'].includes(subcategory)) {
+          return expandQuestions(
+            straatverlichtingKlokken,
+            category,
+            subcategory
+          )
+        }
+
+        if (subcategory === 'verkeersoverlast') {
+          return expandQuestions(verkeersoverlast, category, subcategory)
+        }
+
+        return expandQuestions(
+          wegenVerkeerStraatmeubilair,
+          category,
           subcategory
         )
-          ? straatverlichtingKlokken
-          : wegenVerkeerStraatmeubilair
-
-        return expandQuestions(config, category, subcategory)
       }
 
       case 'wonen':

src/signals/incident/definitions/wizard-step-2-vulaan/afval-thor.test.ts

@@ -0,0 +1,17 @@
+// SPDX-License-Identifier: MPL-2.0
+// Copyright (C) 2024 Gemeente Amsterdam
+import afvalThor from './afval-thor'
+
+describe('definition afvalThor', () => {
+  it('has a defined set of controls', () => {
+    const keys = Object.keys(afvalThor)
+
+    expect(keys).toStrictEqual([
+      'locatie',
+      'dateTime',
+      'extra_afval_handhaving',
+      'extra_afval_handhaving_owner',
+      'extra_afval_handhaving_owner_confirmation',
+    ])
+  })
+})

src/signals/incident/definitions/wizard-step-2-vulaan/afval-thor.ts

@@ -0,0 +1,57 @@
+// SPDX-License-Identifier: MPL-2.0
+// Copyright (C) 2024 Gemeente Amsterdam
+import { inPast } from 'signals/incident/services/custom-validators'
+import { QuestionFieldType } from 'types/question'
+
+import locatie from './locatie'
+
+export const controls = {
+  locatie,
+  dateTime: {
+    meta: {
+      label: 'Wanneer is of was de overlast?',
+    },
+    options: {
+      validators: [inPast, 'required'],
+    },
+    render: QuestionFieldType.DateTimeInput,
+  },
+  extra_afval_handhaving: {
+    meta: {
+      label: 'Welk afval is verkeerd neergezet?',
+      subtitle:
+        'U helpt ons door zoveel mogelijk informatie te geven over het soort afval: huisafval, bedrijfsafval, grofvuil of dozen en papier.',
+      shortLabel: 'Welk afval',
+      pathMerge: 'extra_properties',
+    },
+    render: QuestionFieldType.TextareaInput,
+  },
+  extra_afval_handhaving_owner: {
+    meta: {
+      label:
+        'Weet u wie de eigenaar is van het verkeerd geplaatste afval? Bijvoorbeeld omdat u dat ziet aan een adressticker of iets anders?',
+      shortLabel: 'Welke eigenaar',
+      pathMerge: 'extra_properties',
+      values: {
+        ja: 'Ja',
+        nee: 'Nee',
+      },
+    },
+    options: {
+      validators: ['required'],
+    },
+    render: QuestionFieldType.RadioInput,
+  },
+  extra_afval_handhaving_owner_confirmation: {
+    meta: {
+      ifOneOf: {
+        extra_afval_handhaving_owner: 'ja',
+      },
+      value:
+        'Wij willen graag telefonisch contact met u hierover. Als u dat goed vindt, vul dan uw telefoonnummer in op de volgende pagina.',
+      type: 'info',
+    },
+    render: QuestionFieldType.PlainText,
+  },
+}
+export default controls

src/signals/incident/definitions/wizard-step-2-vulaan/afval.test.ts

@@ -1,16 +1,11 @@
+// SPDX-License-Identifier: MPL-2.0
+// Copyright (C) 2018 - 2024 Gemeente Amsterdam
 import afval from './afval'
 
 describe('definition afval', () => {
   it('has a defined set of controls', () => {
     const keys = Object.keys(afval)
 
-    expect(keys).toStrictEqual([
-      'locatie',
-      'dateTime',
-      'extra_afval_handhaving',
-      'extra_afval_handhaving_owner',
-      'extra_afval_handhaving_owner_confirmation',
-      'extra_afval',
-    ])
+    expect(keys).toStrictEqual(['locatie', 'dateTime', 'extra_afval'])
   })
 })

src/signals/incident/definitions/wizard-step-2-vulaan/afval.ts

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MPL-2.0
-// Copyright (C) 2018 - 2023 Gemeente Amsterdam
+// Copyright (C) 2018 - 2024 Gemeente Amsterdam
 import {
   falsyOrNumberOrNow,
   inPast,
@@ -12,61 +12,14 @@ export const controls = {
   locatie,
   dateTime: {
     meta: {
-      ifOneOf: {
-        subcategory: ['asbest-accu', 'handhaving-op-afval'],
-      },
       label: 'Wanneer was het?',
-      ignoreVisibility: true,
       canBeNull: true,
     },
     options: {
       validators: [falsyOrNumberOrNow, inPast],
     },
     render: QuestionFieldType.DateTimeInput,
   },
-  extra_afval_handhaving: {
-    meta: {
-      ifOneOf: {
-        subcategory: ['asbest-accu', 'handhaving-op-afval'],
-      },
-      label: 'Welk afval is verkeerd neergezet?',
-      subtitle:
-        'U helpt ons door zoveel mogelijk informatie te geven over het soort afval: huisafval, bedrijfsafval, grofvuil of dozen en papier.',
-      shortLabel: 'Welk afval',
-      pathMerge: 'extra_properties',
-    },
-    render: QuestionFieldType.TextareaInput,
-  },
-  extra_afval_handhaving_owner: {
-    meta: {
-      ifOneOf: {
-        subcategory: ['asbest-accu', 'handhaving-op-afval'],
-      },
-      label:
-        'Weet u wie de eigenaar is van het verkeerd geplaatste afval? Bijvoorbeeld omdat u dat ziet aan een adressticker of iets anders?',
-      shortLabel: 'Welke eigenaar',
-      pathMerge: 'extra_properties',
-      values: {
-        ja: 'Ja',
-        nee: 'Nee',
-      },
-    },
-    options: {
-      validators: ['required'],
-    },
-    render: QuestionFieldType.RadioInput,
-  },
-  extra_afval_handhaving_owner_confirmation: {
-    meta: {
-      ifOneOf: {
-        extra_afval_handhaving_owner: 'ja',
-      },
-      value:
-        'Wij willen graag telefonisch contact met u hierover. Als u dat goed vindt, vul dan uw telefoonnummer in op de volgende pagina.',
-      type: 'info',
-    },
-    render: QuestionFieldType.PlainText,
-  },
   extra_afval: {
     meta: {
       ifOneOf: {

src/signals/incident/definitions/wizard-step-2-vulaan/boom-illegale-kap.ts

@@ -1,9 +1,21 @@
-import dateTime from './dateTime'
+// SPDX-License-Identifier: MPL-2.0
+// Copyright (C) 2018 - 2024 Gemeente Amsterdam
+import { inPast } from 'signals/incident/services/custom-validators'
+import { QuestionFieldType } from 'types/question'
+
 import locatie from './locatie'
 
 const boomIllegaleKap = {
   locatie,
-  dateTime,
+  dateTime: {
+    meta: {
+      label: 'Wanneer is of was de overlast?',
+    },
+    options: {
+      validators: [inPast, 'required'],
+    },
+    render: QuestionFieldType.DateTimeInput,
+  },
 }
 
 export default boomIllegaleKap

src/signals/incident/definitions/wizard-step-2-vulaan/bouw-sloop-overlast.ts

@@ -1,9 +1,21 @@
-import dateTime from './dateTime'
+// SPDX-License-Identifier: MPL-2.0
+// Copyright (C) 2018 - 2024 Gemeente Amsterdam
+import { inPast } from 'signals/incident/services/custom-validators'
+import { QuestionFieldType } from 'types/question'
+
 import locatie from './locatie'
 
 const bouwSloopOverlast = {
   locatie,
-  dateTime,
+  dateTime: {
+    meta: {
+      label: 'Wanneer is of was de overlast?',
+    },
+    options: {
+      validators: [inPast, 'required'],
+    },
+    render: QuestionFieldType.DateTimeInput,
+  },
 }
 
 export default bouwSloopOverlast

src/signals/incident/definitions/wizard-step-2-vulaan/overlast-bedrijven-en-horeca.ts

@@ -7,28 +7,26 @@ import locatie from './locatie'
 
 const overlastBedrijvenEnHoreca = {
   locatie,
-
   dateTime: {
     meta: {
       ifOneOf: {
         subcategory: [
           'geluidsoverlast-installaties',
           'geluidsoverlast-muziek',
           'overig-horecabedrijven',
+          'overlast-door-bezoekers-niet-op-terras',
           'overlast-evenementen',
           'overlast-terrassen',
           'stankoverlast',
         ],
       },
-      label: 'Wanneer heeft u de overlast?',
-      canBeNull: true,
+      label: 'Wanneer is of was de overlast?',
     },
     options: {
       validators: [inPast, 'required'],
     },
     render: QuestionFieldType.DateTimeInput,
   },
-
   extra_bedrijven_horeca_frequentie: {
     meta: {
       ifOneOf: {