Skip to content

Conversation

@strijm
Copy link
Collaborator

@strijm strijm commented May 10, 2023

No description provided.

@strijm strijm changed the base branch from develop to verdieping-testen May 10, 2023 16:47
@sanderke sanderke mentioned this pull request May 12, 2023
@sanderke
Copy link
Member

Preview

@mrtn78
Copy link
Collaborator

mrtn78 commented Jun 1, 2023

Voor deze improvement ook nog even nagaan:

  • of de OAS specificatie iets voorschrijft over toegestane http methoden? > Nee, dit geeft geen uitsluitsel
  • een test service/path wenselijk is? > wel wenselijk maar niet realistisch
  • de PROPFIND methode wel wenselijk is om te gebruiken? > Nee
  • we de propfind methode wellicht kunnen gebruiken als verplicht onderdeel? > Nee
  • of we alle servers testen die worden opgesomd of alleen de eerste of de productie server? > Alleen productie

@mrtn78 mrtn78 added the Scope: Klein Kleine wijzigingen met beperkte scope label Jul 6, 2023
@mrtn78
Copy link
Collaborator

mrtn78 commented Jul 6, 2023

Zojuist nog de wijzigingen en de implicaties besproken met @hugoboos . Ook besloten deze te mergen en de branche op te schonen. eventuele aanvullingen of inzichten uit het schrijven van de tests en reviews nemen we mee in de branche https://github.com/Logius-standaarden/API-Design-Rules/tree/verdieping-testen

@mrtn78 mrtn78 merged commit d5d7f98 into verdieping-testen Jul 6, 2023
@mrtn78 mrtn78 deleted the improvement-api-03 branch July 6, 2023 15:25
@mrtn78 mrtn78 added the Status: Gereed Het voorstel is aangenomen en kan worden doorgevoerd. label Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Klein Kleine wijzigingen met beperkte scope Status: Gereed Het voorstel is aangenomen en kan worden doorgevoerd.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants