Merge pull request #210 from maykinmedia/feature/207-indicatie-geheim…

…houding-optional

✨ [#207] changed 'indicatie_geheimhouding' from required to optional

src/openklant/components/klantinteracties/api/tests/test_partijen.py

@@ -88,7 +88,6 @@ def test_create_partij(self):
             "rekeningnummers": [{"uuid": str(rekeningnummer.uuid)}],
             "voorkeursRekeningnummer": {"uuid": str(rekeningnummer.uuid)},
             "soortPartij": "persoon",
-            "indicatieGeheimhouding": True,
             "voorkeurstaal": "ndl",
             "indicatieActief": True,
             "bezoekadres": {
@@ -132,7 +131,7 @@ def test_create_partij(self):
             data["voorkeursRekeningnummer"]["uuid"], str(rekeningnummer.uuid)
         )
         self.assertEqual(data["soortPartij"], "persoon")
-        self.assertTrue(data["indicatieGeheimhouding"])
+        self.assertIsNone(data["indicatieGeheimhouding"])
         self.assertEqual(data["voorkeurstaal"], "ndl")
         self.assertTrue(data["indicatieActief"])
         self.assertEqual(
@@ -188,7 +187,7 @@ def test_create_partij(self):
             self.assertEqual(response_data["rekeningnummers"], [])
             self.assertIsNone(response_data["voorkeursRekeningnummer"])
             self.assertEqual(response_data["soortPartij"], "persoon")
-            self.assertTrue(response_data["indicatieGeheimhouding"])
+            self.assertIsNone(data["indicatieGeheimhouding"])
             self.assertEqual(response_data["voorkeurstaal"], "ndl")
             self.assertTrue(response_data["indicatieActief"])
             self.assertEqual(
@@ -663,7 +662,7 @@ def test_update_partij(self):
             "rekeningnummers": [{"uuid": str(rekeningnummer2.uuid)}],
             "voorkeursRekeningnummer": {"uuid": str(rekeningnummer2.uuid)},
             "soortPartij": "persoon",
-            "indicatieGeheimhouding": False,
+            "indicatieGeheimhouding": None,
             "voorkeurstaal": "ger",
             "indicatieActief": False,
             "bezoekadres": {
@@ -721,7 +720,7 @@ def test_update_partij(self):
             data["voorkeursRekeningnummer"]["uuid"], str(rekeningnummer2.uuid)
         )
         self.assertEqual(data["soortPartij"], "persoon")
-        self.assertFalse(data["indicatieGeheimhouding"])
+        self.assertIsNone(data["indicatieGeheimhouding"])
         self.assertEqual(data["voorkeurstaal"], "ger")
         self.assertFalse(data["indicatieActief"])
         self.assertEqual(

src/openklant/components/klantinteracties/migrations/0016_alter_partij_indicatie_geheimhouding.py

@@ -0,0 +1,23 @@
+# Generated by Django 4.2.11 on 2024-07-31 11:37
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("klantinteracties", "0015_internetaak_afgehandeld_op"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="partij",
+            name="indicatie_geheimhouding",
+            field=models.BooleanField(
+                default=None,
+                help_text="Geeft aan of de verstrekker van partijgegevens heeft aangegeven dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven is dan wordt dit ingevuld als `null`.",
+                null=True,
+                verbose_name="indicatie geheimhouding",
+            ),
+        ),
+    ]

src/openklant/components/klantinteracties/models/partijen.py

@@ -67,8 +67,11 @@ class Partij(APIMixin, BezoekadresMixin, CorrespondentieadresMixin):
         _("indicatie geheimhouding"),
         help_text=_(
             "Geeft aan of de verstrekker van partijgegevens heeft aangegeven dat "
-            "deze gegevens als geheim beschouwd moeten worden."
+            "deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven is "
+            "dan wordt dit ingevuld als `null`."
         ),
+        null=True,
+        default=None,
     )
     voorkeurstaal = models.CharField(
         _("voorkeurstaal"),

src/openklant/components/klantinteracties/openapi.yaml

@@ -4006,8 +4006,10 @@ components:
           description: Geeft aan van welke specifieke soort partij sprake is.
         indicatieGeheimhouding:
           type: boolean
+          nullable: true
           description: Geeft aan of de verstrekker van partijgegevens heeft aangegeven
-            dat deze gegevens als geheim beschouwd moeten worden.
+            dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven
+            is dan wordt dit ingevuld als `null`.
         voorkeurstaal:
           type: string
           description: 'Taal, in ISO 639-2/B formaat, waarin de partij bij voorkeur
@@ -4037,7 +4039,6 @@ components:
       - categorieRelaties
       - digitaleAdressen
       - indicatieActief
-      - indicatieGeheimhouding
       - partijIdentificatoren
       - rekeningnummers
       - soortPartij
@@ -4752,8 +4753,10 @@ components:
           description: Geeft aan van welke specifieke soort partij sprake is.
         indicatieGeheimhouding:
           type: boolean
+          nullable: true
           description: Geeft aan of de verstrekker van partijgegevens heeft aangegeven
-            dat deze gegevens als geheim beschouwd moeten worden.
+            dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven
+            is dan wordt dit ingevuld als `null`.
         voorkeurstaal:
           type: string
           description: 'Taal, in ISO 639-2/B formaat, waarin de partij bij voorkeur

src/openklant/conf/dev.py

@@ -15,6 +15,7 @@
 os.environ.setdefault("DB_PASSWORD", "openklant"),
 
 os.environ.setdefault("ENVIRONMENT", "development")
+os.environ.setdefault("DISABLE_2FA", "yes")
 
 os.environ.setdefault("RELEASE", "dev")
 os.environ.setdefault("LOG_REQUESTS", "no")