[#18] add upn and pt code to product kanaal kenmerken

maykinmedia · Mar 7, 2025 · da2f419 · da2f419
1 parent 1a6d0d8
commit da2f419
Show file tree

Hide file tree

Showing 8 changed files with 32 additions and 16 deletions.
diff --git a/src/notificaties.md b/src/notificaties.md
@@ -22,7 +22,9 @@ De architectuur van de notificaties staat beschreven op <a href="https://github.
 
 **Kenmerken**
 
-* `product_type_id`: Het type van dit product
+* `product_type.id`: 
+* `product_type.uniforme_product_naam`: Uniforme product naam gedefinieerd door de overheid.
+* `product_type.code`: code van het product type.
 
 **Resources en acties**
 

diff --git a/src/open_producten/producten/kanalen.py b/src/open_producten/producten/kanalen.py
@@ -5,5 +5,9 @@
 KANAAL_PRODUCTEN = Kanaal(
     "producten",
     main_resource=Product,
-    kenmerken=("product_type.id",),  # TODO decide on kenmerken
+    kenmerken=(
+        "product_type.id",
+        "product_type.uniforme_product_naam",
+        "product_type.code",
+    ),
 )
diff --git a/src/open_producten/producten/tests/api/test_notifications_send.py b/src/open_producten/producten/tests/api/test_notifications_send.py
@@ -63,6 +63,10 @@ def test_send_notif_create_object(self, mock_task):
             "aanmaakdatum": "2024-02-02T01:00:00+01:00",
             "kenmerken": {
                 "productType.id": data["product_type"]["id"],
+                "productType.uniformeProductNaam": data["product_type"][
+                    "uniforme_product_naam"
+                ],
+                "productType.code": data["product_type"]["code"],
             },
         }
 
@@ -85,6 +89,10 @@ def test_send_notif_update_object(self, mock_task):
             "aanmaakdatum": "2024-02-02T01:00:00+01:00",
             "kenmerken": {
                 "productType.id": data["product_type"]["id"],
+                "productType.uniformeProductNaam": data["product_type"][
+                    "uniforme_product_naam"
+                ],
+                "productType.code": data["product_type"]["code"],
             },
         }
 
@@ -107,6 +115,10 @@ def test_send_notif_partial_update_object(self, mock_task):
             "aanmaakdatum": "2024-02-02T01:00:00+01:00",
             "kenmerken": {
                 "productType.id": data["product_type"]["id"],
+                "productType.uniformeProductNaam": data["product_type"][
+                    "uniforme_product_naam"
+                ],
+                "productType.code": data["product_type"]["code"],
             },
         }
 
@@ -128,6 +140,8 @@ def test_send_notif_delete_object(self, mock_task):
             "aanmaakdatum": "2024-02-02T01:00:00+01:00",
             "kenmerken": {
                 "productType.id": str(self.product_type.id),
+                "productType.uniformeProductNaam": self.product_type.uniforme_product_naam.naam,
+                "productType.code": self.product_type.code,
             },
         }
 

diff --git a/src/open_producten/producten/tests/api/test_product.py b/src/open_producten/producten/tests/api/test_product.py
@@ -81,7 +81,7 @@ def test_create_product(self):
             "product_type": {
                 "id": str(product_type.id),
                 "code": product_type.code,
-                "uniforme_product_naam": product_type.uniforme_product_naam.uri,
+                "uniforme_product_naam": product_type.uniforme_product_naam.naam,
                 "gepubliceerd": True,
                 "toegestane_statussen": ["gereed"],
                 "aanmaak_datum": product_type.aanmaak_datum.astimezone().isoformat(),
@@ -231,7 +231,7 @@ def test_read_producten(self):
                 "product_type": {
                     "id": str(self.product_type.id),
                     "code": self.product_type.code,
-                    "uniforme_product_naam": self.product_type.uniforme_product_naam.uri,
+                    "uniforme_product_naam": self.product_type.uniforme_product_naam.naam,
                     "toegestane_statussen": ["gereed"],
                     "gepubliceerd": True,
                     "aanmaak_datum": self.product_type.aanmaak_datum.astimezone().isoformat(),
@@ -255,7 +255,7 @@ def test_read_producten(self):
                 "product_type": {
                     "id": str(self.product_type.id),
                     "code": self.product_type.code,
-                    "uniforme_product_naam": self.product_type.uniforme_product_naam.uri,
+                    "uniforme_product_naam": self.product_type.uniforme_product_naam.naam,
                     "toegestane_statussen": ["gereed"],
                     "gepubliceerd": True,
                     "aanmaak_datum": self.product_type.aanmaak_datum.astimezone().isoformat(),
@@ -290,7 +290,7 @@ def test_read_product(self):
             "product_type": {
                 "id": str(product_type.id),
                 "code": product_type.code,
-                "uniforme_product_naam": product_type.uniforme_product_naam.uri,
+                "uniforme_product_naam": product_type.uniforme_product_naam.naam,
                 "toegestane_statussen": ["gereed"],
                 "gepubliceerd": True,
                 "aanmaak_datum": "2025-12-31T01:00:00+01:00",

diff --git a/src/open_producten/producttypen/serializers/thema.py b/src/open_producten/producttypen/serializers/thema.py
@@ -11,7 +11,7 @@
 
 class NestedProductTypeSerializer(serializers.ModelSerializer):
     uniforme_product_naam = serializers.SlugRelatedField(
-        slug_field="uri", queryset=UniformeProductNaam.objects.all()
+        slug_field="naam", queryset=UniformeProductNaam.objects.all()
     )
 
     class Meta:

diff --git a/src/open_producten/producttypen/tests/api/test_thema.py b/src/open_producten/producttypen/tests/api/test_thema.py
@@ -384,7 +384,7 @@ def test_read_product_type(self):
             {
                 "id": str(product_type.id),
                 "code": product_type.code,
-                "uniforme_product_naam": product_type.uniforme_product_naam.uri,
+                "uniforme_product_naam": product_type.uniforme_product_naam.naam,
                 "gepubliceerd": True,
                 "toegestane_statussen": [],
                 "aanmaak_datum": product_type.aanmaak_datum.astimezone().isoformat(),

diff --git a/src/producten-openapi.yaml b/src/producten-openapi.yaml
@@ -425,8 +425,7 @@ components:
           description: Lijst van keywords waarop kan worden gezocht.
         uniforme_product_naam:
           type: string
-          format: uri
-          description: Uri naar de UPN definitie.
+          description: Uniforme product naam
         toegestane_statussen:
           type: array
           items:
@@ -467,9 +466,8 @@ components:
           description: Lijst van keywords waarop kan worden gezocht.
         uniforme_product_naam:
           type: string
-          format: uri
           minLength: 1
-          description: Uri naar de UPN definitie.
+          description: Uniforme product naam
         toegestane_statussen:
           type: array
           items:

diff --git a/src/producttypen-openapi.yaml b/src/producttypen-openapi.yaml
@@ -3584,8 +3584,7 @@ components:
           description: Lijst van keywords waarop kan worden gezocht.
         uniforme_product_naam:
           type: string
-          format: uri
-          description: Uri naar de UPN definitie.
+          description: Uniforme product naam
         toegestane_statussen:
           type: array
           items:
@@ -3626,9 +3625,8 @@ components:
           description: Lijst van keywords waarop kan worden gezocht.
         uniforme_product_naam:
           type: string
-          format: uri
           minLength: 1
-          description: Uri naar de UPN definitie.
+          description: Uniforme product naam
         toegestane_statussen:
           type: array
           items: