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

2242 Voeg property inhoudIsVervallen toe aan documenten #2421

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

HenriKorver
Copy link
Collaborator

@HenriKorver HenriKorver commented Mar 4, 2024

Naar aanleiding van dit voorstel.

@HenriKorver HenriKorver requested a review from hdksi March 4, 2024 17:30
@hdksi
Copy link
Collaborator

hdksi commented Mar 5, 2024

Mooi!

  • De beschrijving van property in natuurlijke taal begint volgens conventie steeds met een kleine letter.
    afbeelding.
  • Regel dat PUT en PATCH operaties op enkelvoudige informatieobjecten met status defintief niet zijn toegestaan vervalt.
    afbeelding
  • Regel drc-005 (sorry, mijn fout door vermelding onjuiste regel) vervalt niet in deze versie.
  • Nog een changelog voor bovenaan de OAS toevoegen.
  • Kunnen we bij de property inhoudIsVervallen een opmerking toevoegen over het feit dat deze nieuw is in versie 1.5.0?

@HenriKorver
Copy link
Collaborator Author

HenriKorver commented Mar 5, 2024

Kunnen we bij de property inhoudIsVervallen een opmerking toevoegen over het feit dat deze nieuw is in versie 1.5.0?

Kan wel, maar niet gedaan. De toevoeging van dit nieuwe attribuut staat al vermeld in de changelog bovenaan de OAS. Verder zijn al je opmerkingen verwerkt.

@hdksi
Copy link
Collaborator

hdksi commented Mar 7, 2024

Bij nader inzien ontbreekt in de spec zo te zien de property inhoudIsVervallen nog bij de PATCH-operatie op 'enkelvoudiginformatieobjecten'.

@HenriKorver
Copy link
Collaborator Author

Dank, goed gezien! Ik heb zojuist deze property ook toegevoegd aan de PATCH-operatie.

@HenriKorver HenriKorver merged commit a3b666e into master Mar 12, 2024
@HenriKorver HenriKorver deleted the 2242_drc_inhoudIsVervallen branch March 12, 2024 14:41
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