Skip to content
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

ZTC: afwijkende attribuutnaam voor zaaktypeidenitificatie en datumgeldigheid in response resultaattypen #2473

Open
johannesbattjes opened this issue Sep 12, 2024 · 8 comments

Comments

@johannesbattjes
Copy link

johannesbattjes commented Sep 12, 2024

Bij Resultaattypen is de naamgeving van zaaktypeidentificatie en datumgeldigheid anders dan bij andere objecttypen:
image
Voorstel: naamgeving van deze twee velden aanpassen bij resultaattypen

@HenriKorver
Copy link
Collaborator

Eens, maar helaas is deze fix niet backwards compatible. Dus ik hoop dat anderen het nog niet geïmplementeerd hebben.

@joeribekker, @sergei-maertens: Hebben jullie dit al geïmplementeerd in OpenZaak?

@sergei-maertens
Copy link
Collaborator

merk op dat dit niet enkel om server implementaties gaat, maar ook clients hier last van ondervinden

ik ben zelf niet direct meer betrokken bij de Open Zaak ontwikkeling, dus kan je in de toekomst mij niet meer taggen met Open Zaak-specifieke vragen, dat scheelt weer wat notificaties in mijn inbox 😛

@HenriKorver
Copy link
Collaborator

Is @joeribekker nog wel het contactpunt voor OpenZaak, of is dat ook iemand anders geworden?

@michielverhoef
Copy link
Collaborator

Eens, maar helaas is deze fix niet backwards compatible. Dus ik hoop dat anderen het nog niet geïmplementeerd hebben.

Dit is dan een bugfix, die zou ik zo snel mogelijk doorvoeren. In ieder geval contact opnemen met alle (bekende) partijen dat deze fix er aan zit te komen en deze uitbrengen. Desnoods trek je de versie met de fout er in terug. Het is een beetje een paardenmiddel maar dan blijft de pijn beperkt.

@sergei-maertens
Copy link
Collaborator

Is @joeribekker nog wel het contactpunt voor OpenZaak, of is dat ook iemand anders geworden?

Jawel, maar ik weet uit ervaring dat zijn mailbox ontploft dus de kans dat hij deze notificaties ziet en erop reageert is... klein. Waarschijnlijk kan je best @annashamray taggen, maar dan liefst ook even de vragen in het Engels stellen 😉

@sergei-maertens
Copy link
Collaborator

Eens, maar helaas is deze fix niet backwards compatible. Dus ik hoop dat anderen het nog niet geïmplementeerd hebben.

Dit is dan een bugfix, die zou ik zo snel mogelijk doorvoeren. In ieder geval contact opnemen met alle (bekende) partijen dat deze fix er aan zit te komen en deze uitbrengen. Desnoods trek je de versie met de fout er in terug. Het is een beetje een paardenmiddel maar dan blijft de pijn beperkt.

Ha Michiel, ik snap deze redenering hoor! Echter zou ik hier iets voorzichtiger mee omgaan:

  1. Het is geen bugfix in de zin dat de huidige vorm het onmogelijk maakt om het te gebruiken. Het is vooral een schoonheidsfout
  2. Het "zomaar" intrekken van versies staat niet zo goed - bij mijn weten is er geen notificatie-mechanisme en kom je als leverancier eigenlijk altijd maar op een harde en onvriendelijke manier erachter dat een versie ingetrokken is, of een bugfix toch breaking changes bevat. Ik zou dit liever niet zien gebeuren, maar uitstellen tot een major versie wanneer er toch andere breaking changes doorgevoerd zullen worden.

Andere wijzigingen zijn uitgesteld of via workarounds doorgevoerd met "backwards compatibility" als argument/in het achterhoofd - voor mij als ontwikkelaar drukt dat uit dat ik hoge stabiliteit kan verwachten van de 1.x versies. Ik ben bang dat als je dit als breaking bugfix gaat doorvoeren, dat het vertrouwen in stabiliteit ondermijnt.

@michielverhoef
Copy link
Collaborator

Ah, ik had niet gezien dat dit probleem al in eerdere versies speelt. Dan kun je inderdaad niet zomaar versies terug gaan trekken. Ik dacht dit alleen in de laatste versie voorkwam.

Een oplossing kan zijn een attribuut toe te voegen wat wel overeen komt met de andere naamgevingsconventie en het bestaande attribuut deprecated te verklaren. Na verloop van tijd kan met de volgende major versie het oude attribuut dan komen te vervallen. Dan moeten er wel wat dingen beschreven worden zodat beide attributen altijd dezelfde waarde bevatten.

@HenriKorver
Copy link
Collaborator

HenriKorver commented Oct 10, 2024

Mooi, dan zijn we het allemaal met elkaar eens en is de oplossing duidelijk voor de volgende release(s). Dan kan ik deze bug labelen als een "known issue".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants