FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Descriptions of path fields and form fields default same value by ajant · Pull Request #145 · core-api/python-client · GitHub

This repository was archived by the owner on Mar 18, 2019. It is now read-only.
/ python-client Public archive

Descriptions of path fields and form fields default same value - #145

Open
ajant wants to merge 1 commit into
core-api:masterfrom
ajant:master
Open

Descriptions of path fields and form fields default same value#145
ajant wants to merge 1 commit into
core-api:masterfrom
ajant:master

Conversation

ajant commented Aug 11, 2017

Copy link
Copy Markdown

… fields defaults to "". This results in inconsistent presentation. This commit changes path fields default to "".

Example before change:
"/example-movies/v1/{id}/": {"put": {"description": "Updates the movie with passed movie id.", "parameters": [{"description": null, "required": true, "type": "string", "name": "id", "in": "path"}, {"description": "", "required": true, "type": "string", "name": "title", "in": "formData"}, {"description": "", "required": true, "type": "integer", "name": "rating", "in": "formData"}], "tags": ["example-movies"], "summary": "Updates the movie with passed movie id.", "operationId": "example-movies_v1_update", "consumes": ["application/x-www-form-urlencoded"], "responses": {"200": {"description": ""}}},
Example after change:
"/example-movies/v1/{id}/": {"put": {"description": "Updates the movie with passed movie id.", "parameters": [{"description": "", "required": true, "type": "string", "name": "id", "in": "path"}, {"description": "", "required": true, "type": "string", "name": "title", "in": "formData"}, {"description": "", "required": true, "type": "integer", "name": "rating", "in": "formData"}], "tags": ["example-movies"], "summary": "Updates the movie with passed movie id.", "operationId": "example-movies_v1_update", "consumes": ["application/x-www-form-urlencoded"], "responses": {"200": {"description": ""}}}
Description of id field changed to be consistent with title and rating fields' descriptions.

… fields defaults to "". This results in inconsistent presentation. This commit changes path fields default to "".

Example before change:
"/example-movies/v1/{id}/": {"put": {"description": "Updates the movie with passed movie id.", "parameters": [{"description": null, "required": true, "type": "string", "name": "id", "in": "path"}, {"description": "", "required": true, "type": "string", "name": "title", "in": "formData"}, {"description": "", "required": true, "type": "integer", "name": "rating", "in": "formData"}], "tags": ["example-movies"], "summary": "Updates the movie with passed movie id.", "operationId": "example-movies_v1_update", "consumes": ["application/x-www-form-urlencoded"], "responses": {"200": {"description": ""}}},
Example after change:
"/example-movies/v1/{id}/": {"put": {"description": "Updates the movie with passed movie id.", "parameters": [{"description": "", "required": true, "type": "string", "name": "id", "in": "path"}, {"description": "", "required": true, "type": "string", "name": "title", "in": "formData"}, {"description": "", "required": true, "type": "integer", "name": "rating", "in": "formData"}], "tags": ["example-movies"], "summary": "Updates the movie with passed movie id.", "operationId": "example-movies_v1_update", "consumes": ["application/x-www-form-urlencoded"], "responses": {"200": {"description": ""}}}
Description of id field changed to be consistent with title and rating fields' descriptions.

Copy link
Copy Markdown

Hi, please merge this pull request, I ran into same issue.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL