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

Expand tabel toevoegen #2376

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

michielverhoef
Copy link
Collaborator

expand tabel toegevoegd

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

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)

Copy link
Collaborator

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 | |

Copy link
Collaborator

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.

Copy link
Collaborator Author

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.

Copy link
Collaborator

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.

Copy link
Collaborator

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.

image

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

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

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

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

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.

Copy link
Collaborator

@HenriKorver HenriKorver left a 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

@michielverhoef michielverhoef changed the base branch from master to develop November 28, 2023 11:04
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.

None yet

2 participants