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

Made JoinData spec comparable to the DL draft; tagged JD endpoints with new/existing compared to draft #3

Merged
merged 33 commits into from
Apr 25, 2019

Conversation

alamers
Copy link
Collaborator

@alamers alamers commented Apr 16, 2019

No description provided.

Copy link
Collaborator

@erwinspeybroeck erwinspeybroeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Arjan,

Ik zie/denk dat je :

  • Geconverteerd hebt naar Open api 3 - swagger ziet er goed uit
  • Resources als jsons hebt gedefinieerd – aparte json files
  • Tags hebt aangebracht bij de API operaties (New in draft, New in JD en Existing in Draft)

Heb er eens doorgescand en op eerste zicht (zonder diepgang) lijkt het me goed.
Om te bekijken of de tags kloppen heb ik meer tijd nodig ... (lijkt me het belangrijkste?)

@alamers alamers merged commit 6374ba0 into adewg:master Apr 25, 2019
alamers pushed a commit that referenced this pull request Jun 21, 2019
cookeac pushed a commit that referenced this pull request Aug 15, 2019
cookeac pushed a commit that referenced this pull request Dec 4, 2019
erwinspeybroeck pushed a commit that referenced this pull request Dec 7, 2019
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.

2 participants