Skip to content

Commit

Permalink
✨ [#207] changed 'indicatie_geheimhouding' from required to optional
Browse files Browse the repository at this point in the history
  • Loading branch information
bart-maykin committed Jul 31, 2024
1 parent 81f16aa commit b3a05e6
Show file tree
Hide file tree
Showing 5 changed files with 38 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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": {
Expand Down Expand Up @@ -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(
Expand Down Expand Up @@ -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(
Expand Down Expand Up @@ -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": {
Expand Down Expand Up @@ -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(
Expand Down
Original file line number Diff line number Diff line change
@@ -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",
),
),
]
5 changes: 4 additions & 1 deletion src/openklant/components/klantinteracties/models/partijen.py
Original file line number Diff line number Diff line change
Expand Up @@ -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"),
Expand Down
9 changes: 6 additions & 3 deletions src/openklant/components/klantinteracties/openapi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -4037,7 +4039,6 @@ components:
- categorieRelaties
- digitaleAdressen
- indicatieActief
- indicatieGeheimhouding
- partijIdentificatoren
- rekeningnummers
- soortPartij
Expand Down Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions src/openklant/conf/dev.py
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand Down

0 comments on commit b3a05e6

Please sign in to comment.