Code Monkey home page Code Monkey logo

Comments (16)

MelvLee avatar MelvLee commented on August 22, 2024 1

@kad-tromps, er ligt een design keuze hieronder. Dit is niet bekend bij jou, waardoor het niet duidelijk is waarom er hier voor een array als type is gekozen.
Als een provider links teruggeeft, dan kan hij kiezen of hij de links templated of niet teruggeeft. Templated is vooral handig als er veel subresources zijn. Als er maar enkele (1, 2 of 3) subresources zijn, is het (denken we) makkelijker om niet templated terug te geven. We willen dus beide situaties ondersteunen, en dat is de reden waarom we gekozen hebben om de links altijd als array types te definieren.

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

Bijvoorbeeld /wozobjecten/000500055046 geeft in _links:

        "adresseerbaarObjecten": {
            "href": "https://api.bag.test.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten/{adresseerbaarObjectIdentificaties}",
            "templated": true
        },
        "panden": {
            "href": "https://api.bag.test.kadaster.nl/esd/huidigebevragingen/v1/panden/{pandIdentificaties}",
            "templated": true
        }

Volgens schema moeten dit arrays zijn.

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

Bijvoorbeeld /wozobjecten/000500055044 bevat geen waarden voor belanghebbendeGebruiker.burgerservicenummer. Toch is de link als template opgenomen:

        "belanghebbendeGebruiker": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeGebruiker_burgerservicenummer}",
            "templated": true
        },

Ook is er een link kadastraalOnroerendeZaken opgenomen, terwijl geen kadastraalOnroerendeZaakIdentificaties zijn. Idem voor de link panden (er zijn geen pandIdentificaties)

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

de link adres verwijst naar de testomgeving van BAG. Dat is niet de juiste, gebruikers kunnen de BAG productieomgeving gebruiken. De testomgeving wordt door het BAG ontwikkelteam gebruikt om te testen en is dus niet stabiel.

Dus in plaats van
"adres": {
"href": "https://api.bag.test.kadaster.nl/esd/huidigebevragingen/v1/adressen/0200200000054860"
},
is het beter te verwijzen naar
"adres": {
"href": "https://api.bag.kadaster.nl/esd/huidigebevragingen/v1/adressen/0200200000054860"
},

idem bij de link adresseerbaarObjecten en panden

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

de link kadastraalOnroerendeZaken verwijst naar de acceptatieomgeving van de BRK API. Deze wordt door het Haal Centraal team gebruikt om nieuwe versies van de API te testen en is dus niet stabiel. De juiste url voor testen door gebruikers is (volgens de Getting started) https://api.brk.kadaster.nl/esd-eto-apikey/bevragen/v1/

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

@kad-tromps in /wozobjecten/800000051111 is de belanghebbendeEigenaar en de belanghebbendeGebruiker een niet-natuurlijk persoon. Toch verwijst de link-template naar de BRP, dus naar natuurlijk personen:
"belanghebbendeEigenaar": {
"href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeEigenaar_burgerservicenummer}",
"templated": true
},
"belanghebbendeGebruiker": {
"href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeGebruiker_burgerservicenummer}",
"templated": true
}

Voor niet-natuurlijk personen moet de link naar HR verwijzen. Helaas is er (nog) geen (Haal Centraal of andere) HR API beschikbaar waarnaar je zou kunnen verwijzen. Dus in dit geval zou je geen link kunnen opnemen

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

de template voor belanghebbendeEigenaar en belanghebbendeGebruiker gebruikt nu een underscore i.p.v. een punt. Dat is niet in lijn met hoe het in de andere Haal Centraal API's is gedaan. Dat is denk ik niet handig voor gebruikers.

Als dit niet lukt, wellicht niet als template aanbieden (afgezien van eventueel templated base url voor de externe API).

@MelvLee hoe denk jij hierover? Is het een probleem dat in WOZ de template met underscore als separator de verwijzing naar een property opbouwt?

from haal-centraal-woz-bevragen.

MelvLee avatar MelvLee commented on August 22, 2024

Nee, dit is geen probleem. Zowel punt als underscore kunnen worden gebruikt in de naam van de placeholder.

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

Oke dan laten we het zo. Dan moet er alleen wel een specifieke feature worden gemaakt die beschrijft hoe dit voor WOZ werkt (afwijkend van hoe het in common staat). @MelvLee kan jij dat maken?

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

@MelvLee ik heb hiervoor issue #85 gemaakt

from haal-centraal-woz-bevragen.

fsamwel avatar fsamwel commented on August 22, 2024

@kad-tromps soms wordt belanghebbendeEigenaar als volledige url geleverd en belanghebbendeGebruiker templated, en soms andersom. Zit daar logica achter?

/wozobjecten/000500055044 heeft in _links:

        "belanghebbendeEigenaar": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/000000048"
        },
        "belanghebbendeGebruiker": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeGebruiker_burgerservicenummer}",
            "templated": true
        },

/wozobjecten/000500030828 heeft in _links:

        "belanghebbendeEigenaar": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeEigenaar_burgerservicenummer}",
            "templated": true
        },
        "belanghebbendeGebruiker": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/000000012"
        },

N.B. bij 000500055044 is de eigenaar een natuurlijk persoon en is er geen gebruiker, bij 000500030828 is de eigenaar een niet-natuurlijk persoon en de gebruiker een natuurlijk persoon. Het lijkt er haast op dat, wanneer er geen burgerservicenummer is, de templated link wordt geleverd met template voor burgerservicenummer.

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

de template voor belanghebbendeEigenaar en belanghebbendeGebruiker gebruikt nu een underscore i.p.v. een punt. Dat is niet in lijn met hoe het in de andere Haal Centraal API's is gedaan. Dat is denk ik niet handig voor gebruikers.

Als dit niet lukt, wellicht niet als template aanbieden (afgezien van eventueel templated base url voor de externe API).

@MelvLee hoe denk jij hierover? Is het een probleem dat in WOZ de template met underscore als separator de verwijzing naar een property opbouwt?

Ik had deze al besproken met Melvin. De 'punt' is een gereserveerde character in templated fields (staat voor domain expansion, dus als je een lijst variable=[www, url, nl] hebt, zou .variable naar www.ur.nl moeten resolven). Daarom is er gekozen voor een underscore

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

@kad-tromps soms wordt belanghebbendeEigenaar als volledige url geleverd en belanghebbendeGebruiker templated, en soms andersom. Zit daar logica achter?

/wozobjecten/000500055044 heeft in _links:

        "belanghebbendeEigenaar": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/000000048"
        },
        "belanghebbendeGebruiker": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeGebruiker_burgerservicenummer}",
            "templated": true
        },

/wozobjecten/000500030828 heeft in _links:

        "belanghebbendeEigenaar": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/{belanghebbendeEigenaar_burgerservicenummer}",
            "templated": true
        },
        "belanghebbendeGebruiker": {
            "href": "https://www.haalcentraal.nl/haalcentraal/api/brp/ingeschrevenpersonen/000000012"
        },

N.B. bij 000500055044 is de eigenaar een natuurlijk persoon en is er geen gebruiker, bij 000500030828 is de eigenaar een niet-natuurlijk persoon en de gebruiker een natuurlijk persoon. Het lijkt er haast op dat, wanneer er geen burgerservicenummer is, de templated link wordt geleverd met template voor burgerservicenummer.

ja, precies. Aangezien er voor nu alleen een link is voor burgerservice nummer, geven we een template terug als er geen burgerservice nummer is, omdat de belanghebbende geen natuurlijk persoon is, of omdat het veld er niet is. Maar, zoals eerder gesuggesteert, ik denk dat we de links gewoon weg laten als het veld niet beschikbaar is

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

Bijvoorbeeld /wozobjecten/000500055046 geeft in _links:

        "adresseerbaarObjecten": {
            "href": "https://api.bag.test.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten/{adresseerbaarObjectIdentificaties}",
            "templated": true
        },
        "panden": {
            "href": "https://api.bag.test.kadaster.nl/esd/huidigebevragingen/v1/panden/{pandIdentificaties}",
            "templated": true
        }

Volgens schema moeten dit arrays zijn.

@fsamwel Ik had dit eerder met @MelvLee besproken, en daar kwamen we tot de conclusie dat we deze templated willen laten, zodat de links niet te lang worden, moet dit wel blijven? Of moeten we in de array ook alle bijbehorende identificaties geven?

from haal-centraal-woz-bevragen.

melsk-r avatar melsk-r commented on August 22, 2024
  • /wozobjecten/000500055046 geeft in '_links' nu wel arrays terug voor 'adresseerbaarObjecten' en 'panden' (zie gerelateerde post) - opgelost.
  • /wozobjecten/000500055044 geeft in '_links' geen 'belanghebbendeGebruiker' meer terug (zie gerelateerde post) - opgelost.
  • /wozobjecten/800000051111 geeft in '_links' nu geen 'belanghebbendeEigenaar' en 'belanghebbendeGebruiker' meer terug (zie gerelateerde post) - opgelost.
  • /wozobjecten/000500030828 geeft helaas geen resource terug aangezien het niet meer bestaat dus hierop kan ik niet testen (zie gerelateerde post).

Blijven alleen de opmerkingen over m.b.t. de baseurl's. Die zijn gerelateerd met issue 76.

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

Basis URLs voor externe links zijn aangepast in de configuratie

from haal-centraal-woz-bevragen.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.