Develop 1.2 #413
Merged
Develop 1.2 #413
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sync master naar develop
toevoegen adresregels aan adres resource
update develop met wijzigingen in master
minLength en maxLength verwijderen bij zoekparameters zoek en zoekresultaatIdentificatie
terugvoeren wijzigingen in master naar develop
wijziging van master naar develop
…oductie" This reverts commit 2c0c6b7.
master naar develop
Verwijzingen naar common1.2.0 aangebracht
Update goals-canvas.md
Underscores verwijderd
feature voor adresregels conform NEN5825
common.yaml v1.3.0 doorgevoerd
JohanBoer
requested review from
fsamwel,
melsk-r,
MelvLee,
strijm and
CathyDingemanse
fsamwel
approved these changes
Jul 5, 2021
MelvLee
reviewed
Jul 5, 2021
| | [343](https://github.com/VNG-Realisatie/Haal-Centraal-BAG-bevragen/issues/343) | Gebruiker mag niet worden verplicht om Accept-Crs te definieren |\ | ||
| | [360](https://github.com/VNG-Realisatie/Haal-Centraal-BAG-bevragen/issues/360) | Als gebruiker wil ik het samengesteld adres krijgen | | ||
| | [363](https://github.com/VNG-Realisatie/Haal-Centraal-BAG-bevragen/issues/363) | als developer van de BRK API wil ik de korteNaam kunnen leveren bij het adres van een kadastraal object | | ||
| | [364](https://github.com/VNG-Realisatie/Haal-Centraal-BAG-bevragen/issues/364) | toevoegen controle op minimale en maximale lengte invoer zoek | |
There was a problem hiding this comment.
Ik denk dat het handig is om dit beter te beschrijven. Hier staat dat er controle is toegevoegd, dus ik verwacht dat minLength en maxLength is toegevoegd aan de zoek parameter. Maar die kom ik daar niet tegen. Als ik de issue lees, concludeer ik dat de gekozen oplossing is: geen zoek resultaat bij een zoek waarde gelijk aan 0 of groter dan 255. Titel van issue komt niet overeen met gekozen oplossing.
MelvLee
reviewed
Jul 5, 2021
| ZoekResultaatHalCollectie: | ||
| type: object | ||
| description: "Resultaten als lijst." | ||
| properties: | ||
| _links: | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.1.0/api-specificatie/common.yaml#/components/schemas/HalPaginationLinksMetLast' | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.3.0/api-specificatie/common.yaml#/components/schemas/HalPaginationLinksMetLast' | ||
| _embedded: |
There was a problem hiding this comment.
Het is mij niet eerder opgevallen, maar dit zorgt ervoor dat er een class met de naam _embedded wordt gegenereerd
MelvLee
reviewed
Jul 5, 2021
| AdresHalCollectie: | ||
| type: object | ||
| properties: | ||
| _links: | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.1.0/api-specificatie/common.yaml#/components/schemas/HalPaginationLinksMetLast' | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.3.0/api-specificatie/common.yaml#/components/schemas/HalPaginationLinksMetLast' | ||
| _embedded: |
There was a problem hiding this comment.
Zelfde als bij ZoekResultaatHalCollectie. Hier krijgt je class met naam _embedded2
MelvLee
reviewed
Jul 5, 2021
| type: array | ||
| self: | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.1.0/api-specificatie/common.yaml#/components/schemas/HalLink' | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.3.0/api-specificatie/common.yaml#/components/schemas/HalLink' | ||
| AdresseerbaarobjectHalCollectie: | ||
| type: object | ||
| properties: |
There was a problem hiding this comment.
Ook hier heb je een inline object, dat zorgt voor een gegenereerde class met naam _embedded
MelvLee
reviewed
Jul 5, 2021
|
|
||
| PandHalCollectie: | ||
| type: object | ||
| properties: | ||
| _links: | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.1.0/api-specificatie/common.yaml#/components/schemas/HalCollectionLinks' | ||
| $ref: 'https://raw.githubusercontent.com/VNG-Realisatie/Haal-Centraal-common/v1.3.0/api-specificatie/common.yaml#/components/schemas/HalCollectionLinks' | ||
| _embedded: |
|
Verder zie ik aardig wat meldingen mbt het gebruiken van snake-case voor enum values. Dit staat ook in de design decisions. Geldt dit nog? |
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
melsk-r
reviewed
Jul 5, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release 1.2 doorzetten naar de master om release 1.2 aan te maken.