search

melsk-r / HC-BRK-bevragen-issues

0 stars 0 forks source link

Als PO stel ik voor om de Objectlocatie binnenland te mappen naar de adres resource uit de HC BAG API #83

Open melsk-r opened 2 weeks ago

melsk-r commented 2 weeks ago

Originally created by CathyDingemanse (https://github.com/VNG-Realisatie/Haal-Centraal-BRK-bevragen/issues/374):

...zodat de HC API's consistent zijn

De description is al goed!

description: Een ObjectlocatieBinnenland is een adres binnen nederland.
identificatie string
huisletter stringtitle: huisletterURI https://bag.basisregistraties.overheid.nl/doc/begrip/Huisletter [a-zA-Z]
huisnummer integertitle: huisnummerURI https://bag.basisregistraties.overheid.nl/doc/begrip/Huisnummer [1-9][0-9]{0,4}
huisnummerToevoeging stringtitle: huisnummertoevoegingURI https://bag.basisregistraties.overheid.nl/doc/begrip/Huisnummertoevoeging ([a-z,A-Z,0-9])+
inOnderzoek booleantitle: inOnderzoekURI https://bag.basisregistraties.overheid.nl/doc/begrip/InOnderzoek
openbareruimtenaam stringtitle: openbareRuimteNaamminLength: 1
postcode stringtitle: postcodeURI https://bag.basisregistraties.overheid.nl/doc/begrip/Postcode [1-9][0-9][0-9][0-9][A-Z][A-Z]
woonplaatsnaam stringtitle: woonplaatsNaamminLength: 1

Acceptatiecriteria

Definition of done

melsk-r commented 2 weeks ago

This comment originally might have been created by someone else.

@CathyDingemanse dus je wilt dat we openbareRuimteNaam gebruiken i.p.v. straat en woonplaatsnaam i.p.v. woonplaats?

melsk-r commented 2 weeks ago

This comment originally might have been created by someone else.

@CathyDingemanse dus je wilt dat we openbareRuimteNaam gebruiken i.p.v. straat en woonplaatsnaam i.p.v. woonplaats?

Nee juist niet. Conform ADRES RESOURCE uit de bag bevragen api, dus met straat en woonplaats :-) Maar dat heb je vlgs mij gedaan. Ik zat nog in de huidige swagger te kijken voor de descriptions.

melsk-r commented 2 weeks ago

This comment originally might have been created by someone else.

@CathyDingemanse ik was in de war gebracht door de tabel bovenaan dit issue. Dacht even dat je wilde dat het zo zou worden.

Ik heb het nog niet helemaal conform BAG adres gedaan: korteNaam ontbreekt nog.

Echt conform Adres resource zou zijn overerven uit de BAG specs. Dus met een $ref naar de BAG Github. Daarvoor moet er in BAG dan een AdresBeperkt komen, die hier (her)gebruikt wordt, en waar de verschillende identificaties (openbareRuimteIdentificatie, woonplaatsIdentificatie, enz.) en andere extra's (indicatieNevenadres, indicatieGeconstateerd, mogelijkOnjuist) niet in zitten.

Dat (aanpassingen in BAG yaml) lijkt me een mooi klusje voor @JohanBoer