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

Tijdelijk geen gezag toelichting #1875

Merged
merged 9 commits into from
Jan 30, 2025

Conversation

nielskorporaal
Copy link
Collaborator

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:

  • het gezag is niet te bepalen met de toelichting 'toelichting'
  • er is tijdelijk geen gezag met de toelichting 'toelichting'

Nieuwe Als-stap:

  • Als gezag wordt gevraagd van 'naam'

Meerdere namen opnemen gescheiden met komma's is ook mogelijk:

  • Als gezag wordt gevraagd van 'naam1,naam2,naam3'

@@ -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
Copy link
Collaborator

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)

Copy link
Collaborator Author

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.

Copy link
Collaborator

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'
Copy link
Collaborator

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

@fsamwel
Copy link
Collaborator

fsamwel commented Dec 20, 2024

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 nielskorporaal changed the base branch from master to api-specificaties-gezag-toelichting December 20, 2024 10:35
Base automatically changed from api-specificaties-gezag-toelichting to master January 9, 2025 12:01
@fsamwel
Copy link
Collaborator

fsamwel commented Jan 30, 2025

@nielskorporaal er zitten nog conflicten in deze PR waardoor het nog niet gemerged kan worden. Kan jij deze oplossen?

@MelvLee MelvLee force-pushed the tijdelijk-geen-gezag-toelichting branch from c84885d to 538f4b2 Compare January 30, 2025 10:22
@nielskorporaal nielskorporaal merged commit f3c226c into master Jan 30, 2025
4 checks passed
@nielskorporaal nielskorporaal deleted the tijdelijk-geen-gezag-toelichting branch January 30, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants