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

updating swagger parameter descriptions #14

Merged
merged 3 commits into from
Jul 28, 2020
Merged

Conversation

FabiKo117
Copy link
Contributor

marking the types, keys, values params as deprecated
moving default values from deprecated params to filter
adapting some descriptions and default values

marking the types, keys, values params as deprecated
moving default values from deprecated params to filter
adapting some descriptions
@FabiKo117
Copy link
Contributor Author

I just saw that there are also quite some not-related formatting updates. Guess they are related to the change of the package naming.

@bonaparten bonaparten self-requested a review July 28, 2020 06:52
@bonaparten bonaparten self-requested a review July 28, 2020 06:56
Copy link
Contributor

@bonaparten bonaparten left a comment

Choose a reason for hiding this comment

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

Tetxtual info at some endpoints - e.g. /ratio - still need updates considering types, keys, values parameters.

Copy link
Contributor

@bonaparten bonaparten left a comment

Choose a reason for hiding this comment

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

It seems good to me.

@FabiKo117 FabiKo117 merged commit 35a0292 into master Jul 28, 2020
@FabiKo117 FabiKo117 deleted the swagger-params-updates branch July 28, 2020 11:33
@tyrasd tyrasd added the documentation Improvements or additions to documentation label Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants