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

Issues 12, 22, 23, 24 #35

Merged
merged 5 commits into from
Feb 27, 2018
Merged

Issues 12, 22, 23, 24 #35

merged 5 commits into from
Feb 27, 2018

Conversation

cportele
Copy link
Member

@cportele cportele commented Jan 30, 2018

The text, OpenAPI fragments and examples now focus on HTML and JSON.

Resolves #24 (in conjunction with the changes in #25/#29).

The text, OpenAPI fragments and examples now focus on HTML and JSON.

Resolves #19 (together with #25/#29).
@cportele
Copy link
Member Author

@cportele cportele changed the title Issue 19 Issue 24 Jan 30, 2018
@cportele
Copy link
Member Author

Note: I mixed issue numbers, the branch should have been called "issue-24"...

Resolves #12:

* removed f
* added discussion

Resolves #22:

* removed 'resultType' (from Core)

Resolves #23:

* removed startIndex
* clarify that prev is always optional
* clarify count
@cportele cportele changed the title Issue 24 Issues 12, 22, 23, 24 Feb 7, 2018
@cportele
Copy link
Member Author

cportele commented Feb 7, 2018

I have added the updates for issues #12, #22 and #23 to this branch/pull request as they are all somewhat related and parallel editing would have made merging later on more complicated.

@cportele
Copy link
Member Author

cportele commented Feb 7, 2018

Note: openapi.yaml and swagger.yaml eventually need to be adapted to these changes.

@cportele cportele merged commit 469e3bb into master Feb 27, 2018
@cportele cportele deleted the issue-19 branch February 27, 2018 16:00
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.

1 participant