-
Notifications
You must be signed in to change notification settings - Fork 28
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
Expand tabel toevoegen #2376
base: develop
Are you sure you want to change the base?
Expand tabel toevoegen #2376
Conversation
expand teable toegevoegd
| catalogi | 1.3.0 | statustypen | zaaktype | Indirect | catalogi | zaaktypen | | | ||
| catalogi | 1.3.0 | statustypen | catalogus | Indirect | catalogi | catalogussen | | | ||
| catalogi | 1.3.0 | statustypen | eigenschappen | Indirect | catalogi | eigenschappen | | | ||
| catalogi | 1.3.0 | zaakobjecttypen | objecttype | Indirect | catalogi | objecttypen | | |
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.
De rij met het veld "objecttype" kan niet worden ge-expandeerd en moet worden weggelaten. Immers er bestaat geen resource "objecttypen" in de catalogi API.
</span> | ||
|
||
Vanaf versie 1.3.0 is het mogelijk de Catalogi API te expanden. Het expand patroon staat beschreven op [deze pagina](../expand_patroon). In onderstaande tabel staat beschreven welke velden in de Catalogi API geëxpandeerd kunnen worden. Deze velden kunnen vooralsnog alleen indirect geëxpandeerd worden, dus vanuit de Zaken API of Documenten API. Niet rechtstreeks vanuit de Catalogi API zelf. Een volledig overzicht van velden die geëxpandeerd kunnen worden staat op [deze pagina](../expand_patroon) | ||
|
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.
De kolom "Opmerkingen" in onderstaande tabel heeft hier weinig toegevoegde waarde en kan worden weggelaten.
| catalogi | 1.3.0 | informatieobjecttypen | catalogus | Indirect | catalogi | catalogussen | | | ||
| catalogi | 1.3.0 | informatieobjecttypen | zaaktypen | Indirect | catalogi | zaaktypen | | | ||
| catalogi | 1.3.0 | informatieobjecttypen | besluittypen | Indirect | catalogi | besluittypen | | | ||
|
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 zou in de tabel of de drie rijen van de Catalogi API weglaten of alle rijen van de Catalogi API opnemen. Immers je kunt ook in de Documenten API via geneste expand alle resources van de Catalogi API bereiken.
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.
Immers je kunt ook in de Documenten API via geneste expand alle resources van de Catalogi API bereiken.
Vanuit de Documenten API kun je alleen de resources van de Catalogi API benaderen waar de resources van de Documenten API ook daadwerkelijk naar verwijzen. Een ZaaktypeInformatieobjecttype bv. wordt niet gebruikt in de Documenten API en kan dus van daaruit ook niet geëxpandeerd worden. Hetzelfde geldt voor een zaaktype. Dat zijn gegevens die alleen vanuit de Zaken API benaderd kunnen worden.
Ook indirect kunnen deze gegevens niet benaderd worden, er is immers geen relatie vanuit EnkelvoudigInformatieObjecttype naar ZaaktypeInformatieobjecttype. Die relatie ligt andersom, vanuit het ZaaktypeInformatieobjecttype naar het Enkelvoudiginformatieobjecttype.
Zo gezien is het beperken van de vanuit de Documenten API te bereiken gegevens in de Catalogi API dus correct mijns inziens. Hetzelfde geldt voor de Besluiten API.
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.
Een ZaaktypeInformatieobjecttype bv. wordt niet gebruikt in de Documenten API en kan dus van daaruit ook niet geëxpandeerd worden
Ja dat klopt, momenteel kan de resource zaaktypeinformatieobjecttypen
niet bereikt worden via geneste expands. Dat is volgens mij een fout in de OAS en ook een fout in de excel-spreadsheet die nog hersteld moet worden.
Hetzelfde geldt voor een zaaktype
Deze opmerking begrijp ik niet. Immers de "zaaktypen" die bij een informatieobjecttype kunnen wel genest verder geëxpandeerd worden. Bijvoorbeeld:
GET /enkelvoudiginformatieobjecten?expand=informatieobjecttypen.zaaktypen.roltypen
Dat houdt in dat alle rijen met de resource "zaaktypen" ook opgenomen moeten worden, etc.
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.
Dat is volgens mij een fout in de OAS en ook een fout in de excel-spreadsheet die nog hersteld moet worden.
Het attribuut "informatieobjecttypen" is niet van het type URI en heeft bovendien geen beschrijving. Volgens mij zou dit een verwijzing moeten zijn naar de resource zaaktypeinformatieobjecttypen
en dan kan deze resource wel bereikt worden via nesting in de expand.
| zaken | 1.5.0 | statussen | gezetdoor | Ja | zaken | rollen | | | ||
| zaken | 1.5.0 | statussen | zaakinformatieobjecten | Ja | zaken | zaakinformatieobjecten | | | ||
| zaken | 1.5.0 | zaakobjecten | zaakobjecttype | Ja | zaken | zaakobjecttypen | | | ||
| zaken | 1.5.0 | zaakverzoeken | zaak | Ja | zaken | zaken | Niet te expanden via zaak | |
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.
Van rij 79 t/m 91 (dus van "zaken::zaakeigenschappen" t/m "zaakverzoeken") moet de Expand-kolom de waarde "Indirect" hebben.
| catalogi | 1.3.0 | statustypen | zaaktype | Indirect | catalogi | zaaktypen | | | ||
| catalogi | 1.3.0 | statustypen | catalogus | Indirect | catalogi | catalogussen | | | ||
| catalogi | 1.3.0 | statustypen | eigenschappen | Indirect | catalogi | eigenschappen | | | ||
| catalogi | 1.3.0 | zaakobjecttypen | objecttype | Indirect | catalogi | objecttypen | | |
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.
Het veld "objecttype" kan niet ge-expandeerd worden en moet worden weggelaten.
| catalogi | 1.3.0 | statustypen | zaaktype | Indirect | catalogi | zaaktypen | | | ||
| catalogi | 1.3.0 | statustypen | catalogus | Indirect | catalogi | catalogussen | | | ||
| catalogi | 1.3.0 | statustypen | eigenschappen | Indirect | catalogi | eigenschappen | | | ||
| catalogi | 1.3.0 | zaakobjecttypen | objecttype | Indirect | catalogi | objecttypen | | |
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.
Zoals eerder opgemerkt: het veld objecttype kan niet ge-expandeerd worden en moet worden weggelaten.
| zaken | 1.5.0 | statussen | gezetdoor | Ja | zaken | rollen | | | ||
| zaken | 1.5.0 | statussen | zaakinformatieobjecten | Ja | zaken | zaakinformatieobjecten | | | ||
| zaken | 1.5.0 | zaakobjecten | zaakobjecttype | Ja | zaken | zaakobjecttypen | | | ||
| zaken | 1.5.0 | zaakverzoeken | zaak | Ja | zaken | zaken | Niet te expanden via zaak | |
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.
Van rij 512 t/m 524 (dus van "zaken::zaakeigenschappen" t/m "zaakverzoeken") moet de Expand-kolom de waarde "Indirect" hebben.
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.
Zie mijn in-line feedback
expand tabel toegevoegd