-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Als gebruiker wil ik zaakobjecten kunnen koppelen conform het zaaktype plus zaakobjecttypen #1848
Comments
Goed voorstel. Ik zie echter niet meteen de noodzaak voor een volledig redesign en rework. Volgens mij kan dit prima opgelost worden door in de Catalogi API de uit het RGBZ-bekende resource |
Zaakobjecttype staat al op de nominatie meegenomen te worden in de nieuwe versie van de Catalogi API en deze aanpassing neem ik ook gelijk mee. |
@MarcoKlerks @hdksi bij deze de userstory naar aanleiding van ons gesprek van vrijdag: #1871 |
De resource Zaakobjecttype isopgenomen in het ontwerp van de Catalogi API: https://app.swaggerhub.com/apis/michielverhoef/Catalogussen_API/1.2.0-alpha#/ |
Top! Maar hiermee heb je in feite niet deze user story opgelost, maar #1846 . Deze user story vergt een aanpassing van de Zaken API, zodat jouw wijzigingen in de Catalogi API ook juist gebruikt worden bij het creëren / updaten van zaken en zaakobjecten. Ik denk dus dat je in de backlog #1846 op Done wil zetten en deze user story weer terug op In Progress. |
Je hebt gelijk, ik had al zo'n idee dat ik het verkeerde issue te pakken had. |
M.i. koppel je geen zaakobjecten maar koppel je objecten waarbij zaakobjecten de 'koppeltabel' is oftewel de manier waarmee je de koppeling legt. M.i. verstandig om dit goed uit elkaar te houden. |
In de resource Zaakobjecten (Zaken API) moet een verwijzing naar het Zaakobjecttype (Catalogi API) worden opgenomen. Zaakobjecttype is opgenomen in de Catalogi API 1.2-alpha https://catalogi-api.test.vng.cloud/api/v1/schema/#operation/zaakobjecttype_read |
@michielverhoef Welke URL verwijzing moet er gemaakt worden als het objecttype wel uit RGBZ/RSGB komt, bijvoorbeeld een pand? Gezien het een verplicht veld is... |
Hmm, het vervelende is dat objecttype een soort van zaakobjecttype is, alleen met beperkingen. Het doel van Zaakobjecttype in de Catalogi API was de volledigheid van zaakregistraties beter te kunnen garanderen (zie #1846). Met alleen een veld kan dat niet afgedwongen worden, zelfs al is dat veld verplicht. Want als het zaakobject niet aangemaakt wordt is er geen controle mogelijk. Als we zaakobjecttype in de Catalogi API gaan regelen worden de velden objectype en objecttypeOverige overbodig volgens mij. |
Ja volgens mij zouden de attributen Objecttype, ObjecttypeOverige en ObjectTypeOverigeDefinitie uit de zaken api verplaatst moeten worden naar zaakobjecttype in de Catalogi API. |
Dit moet in ieder geval:
Om de nieuwe versie van de Zaken API toch backwards compatible te houden stel ik voor:
https://catalogi-api.test.vng.cloud/api/v1/schema/#operation/zaakobjecttype_read |
Eens.
Eerst even zorgen dat we geen dingen door elkaar halen: Gegeven onderstaande afbeelding: Objecttype (de URL) in de catalogi API zou denk ik zoals eerder vermeld niet verplicht mogen zijn zolang we niet naar een schema kunnen verwijzen voor de RSGB objecttypen. Mijn voorstel zou eigenlijk zijn om objecttype, ObjecttypeOverige en ObjectTypeOverigeDefinitie zoals ze nu in de zaken-API gebruikt worden, 1 op 1 over te zetten naar de catalogi API waarbij in de zaken API alleen een attribuut over blijft om de verwijzing naar de objecttype resource in de catalogi te kunnen maken (URI). |
De bedoeling is, vanuit het RGBZ geredeneerd, om in de Zaken-API een zaak aan nul of meer objecten te kunnen relateren. Aangezien een dergelijk object aan meerdere zaken gerelateerd zou kunnen zijn, ontstaat een n:m-relatie tussen zaak en object. En aangezien die relatie eigenschappen heeft, is er een relatieklasse Zaakobject. Maar daar draait het dus niet om, het gaat om de relatie van een zaak naar objecten. En die objecten zijn van bepaalde objecttypen (Pand, Persoon, Wegdeel, Kadastraal object etc.). Van welke objecttypen er objecten aan een zaak gekoppeld kunnen worden, zou geregeld moeten zijn in de Catalogus-API, zie mijn opmerking van vandaag bij issue #1846 Of een dergelijk object al dan niet een RSGB-object betreft, is m.i. in een CG-gegevenslandschap niet relevant. |
Je bedoelt dus dat de relatieklasse Zaakobjecttype in de Catalogi API wel moet bestaan maar dat de zaak een attribuut objecten kent met daarin één of meer verwijzingen naar relevante objecten (RSGB of anderszins)? |
Wat het qua API's betekent, kan ik niet helemaal overzien. Hoe dan ook, voor de Catalogus-API is vooral de resource Objecttype van belang. Deze heeft een n:m-relatie met Zaaktype. Deze relatie heeft eigenschappen: de Relatieomschrijving, de relaties naar Resultaattype en Statustype en de materiele historie. Dus dat neigt naar een relatieklasse, zou ik zeggen. I.i.g. modelmatig. |
@ArjanKloosterboer dat is feitelijk wat er ook gedaan gaat worden in de resource Zaakobject. Momenteel staan daar nog attributen in om het soort object te omschrijven. Dit omdat er in de Catalogi API nog geen Zaakobjecttype bestond. Het groepsattribuut objectidentificatie is analoog aan de StUF matchinggegevens bedoeld om een (BAG?) object te identificeren waarvoor nog geen API beschikbaar is. Tzt. komt dit attribuut te vervallen. De attributen objecttype, objecttypeOverige en objecttypeOverigeDefinitie worden verplaatst naar Zaakobjecttype uit de Catalogi API. OLm backwards compatible te blijven laten we deze attributen nog in de huidige Zaakobject resource maar worden deze deprercated gemaakt. |
… zodat de aan de zaak gekoppelde (zaak-)objecten overeenkomen met wat op het zaaktype geconfigureerd is. Dit helpt om ervoor te zorgen dat alleen relevante en alle relevante (zaak-)objecten worden gekoppeld.
Let op! User story #1846 is voorwaardelijk voor deze!
Bij het aanmaken van een ‘zaak’ moet de gebruiker een ‘zaaktype’ koppelen. Als de gebruiker een ‘zaakobject’ wil aanmaken, dan moet de gebruiker:
Dit vraagt om een volledig redesign en rework van de huidige zaakobject-services. Er zal dus goed nagedacht moeten worden over backward compatibility of migratie. De nieuwe zaakobject-services zijn wel flexibeler en minder onderhoudsgevoelig, doordat de enumeratie (‘objectType’) vervangen wordt door een verwijzing en er niet op grond van de gekozen optie in de enumeratie andere velden (onder ‘objectIdentificatie’) ondersteund hoeven te worden.
The text was updated successfully, but these errors were encountered: