-
Notifications
You must be signed in to change notification settings - Fork 22
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
Tijdelijk geen gezag toelichting #1875
Conversation
specificatie/resolved/openapi.yaml
Outdated
@@ -7,7 +7,7 @@ info: | |||
Gegevens die er niet zijn of niet actueel zijn krijg je niet terug. Had een persoon bijvoorbeeld een verblijfstitel die nu niet meer geldig is, dan wordt die verblijfstitel niet opgenomen. In partners wordt alleen de actuele of de laatst ontbonden partner geleverd. | |||
|
|||
Zie de [Features overzicht](https://brp-api.github.io/Haal-Centraal-BRP-bevragen/v2/features-overzicht) en [Getting started](https://brp-api.github.io/Haal-Centraal-BRP-bevragen/v2/getting-started) voor nadere toelichting. | |||
version: 2.3.0 | |||
version: 2.4.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waarom wordt de nieuwe versie 2.4.1? Waar komt die patch versie vandaan?
De API zit nu op 2.4.0 (dat is de release met gezag niet te bepalen toelichting), specs zaten op 2.3.0 (software versie en specificatie versie lopen uit elkaar). Dan wordt de nieuwe softwareversie 2.5.0. Is het niet handig de specificatieversie gelijk te maken met het versienummer - minor versie - zoals we die ook in de releasenotes zetten?
(patchversies van de software wijzigen zonder wijziging in de interface specificaties)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
edit: base branch aangepast naar api-specificaties-gezag-niet-te-bepalen
Dat komt omdat (voorlopige) versie 2.4.0 de toelichting bij GezagNietTeBepalen bevat en deze aanpassingen zijn daarop gebaseerd maar zijn later ontwikkeld. We kunnen deze aanpassingen eventueel in een nieuwe minor versie releasen of eventueel opnemen in de releasenotes van 2.4.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR #1831 (toelichting gezag niet te bepalen) maakt versie 2.4.0. Ik zei het hierboven dus verkeerd. 2.3.0 is nog oude(re) versie. Dus toevoegen toelichting tijdelijk geen gezag moet m.i. worden v2.5.0.
|
||
Regel: Een toelichting en de burgerservicenummer van het kind worden geleverd als het gezag van een kind dat verblijft op het gevraagde adres niet kan worden bepaald | ||
|
||
Scenario: 'burgerservicenummer (01.20)' wordt gevraagd met 'burgerservicenummer' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ik begrijp de titel van dit scenario niet
in deze PR wordt o.a. genereervariant, gbagenereervanriant verwijderd en resolve toegevoegd/verplaatst. Maar dat is eerder ook al gedaan, zo wat er nu in master staat. Kijk ik wel naar het verschil tussen wat nu in master staat en wat in de PR wordt gewijzigd? In resolved staat nu ook een json variant van de specificaties, maar die wordt in deze PR niet vernieuwd. Dat had wel gemoeten, want toelichting is toegevoegd bij TijdelijkGeenGezag. |
@nielskorporaal er zitten nog conflicten in deze PR waardoor het nog niet gemerged kan worden. Kan jij deze oplossen? |
c84885d
to
538f4b2
Compare
De OAS componenten m.b.t. TijdelijkGeenGezag zijn bijgewerkt en versie-nr is opgehoogd naar 2.4.1. De ci is aangepast zodat de gezag features geautomatiseerd uitgevoerd kunnen worden.
Nieuwe Gegeven-stappen:
Nieuwe Als-stap:
Meerdere namen opnemen gescheiden met komma's is ook mogelijk: