Tcm 5257 summarize issues with swagger api documentation for creating sdk #246
Conversation
DmitryMasley
requested review from
junky,
dimitrystd,
msogin,
DmitryBochorishvili,
klegins-smartling and
Jackalx
| @@ -114,7 +114,7 @@ components: | |||
| content: | |||
| application/json: | |||
| schema: | |||
| $ref: '#/components/schemas/Error400Response' | |||
| $ref: '../api_common.yaml#/components/schemas/Error400Response' | |||
There was a problem hiding this comment.
I assume api_common.yaml is a most common description of the response
| defaultLocaleId: | ||
| type: string | ||
| description: Smartling's default locale that corresponds to languageId | ||
| properties: |
There was a problem hiding this comment.
object should have properties attribute
| @@ -8,10 +8,6 @@ x-paths: | |||
| tags: | |||
| - File Machine Translations (MT) | |||
| operationId: mtFile | |||
| consumes: | |||
There was a problem hiding this comment.
consumes and produces no longer exist in openapi v3
There was a problem hiding this comment.
consumes was replaced by requestBody and produces described in responses
| @@ -45,6 +45,8 @@ components: | |||
| schemas: | |||
| GetGlossaryEntriesByFilterCommandPTO: | |||
| type: object | |||
| required: | |||
| - entryState | |||
There was a problem hiding this comment.
required fields of the object described in the separate array field required
|
|
||
| externalDocs: | ||
| description: Smartling Help Center | ||
| url: 'https://help.smartling.com' | ||
|
|
||
| security: | ||
| - BearerAuth: [] |
There was a problem hiding this comment.
I assume all the API is using BearerAuth. Please, correct it if it is wrong.
| @@ -3827,85 +3830,85 @@ paths: | |||
| # Issues API endpoints | |||
| # | |||
| /issues-api/v2/dictionary/issue-states: | |||
| $ref: './spec/issues/dictionaries.yaml#/x-paths/dictionary_issues_states' | |||
There was a problem hiding this comment.
reference relative to the file, rather than the root of the project.
| @@ -11811,7 +11832,7 @@ components: | |||
| description: The text to translate | |||
| example: Text to translate | |||
| type: string | |||
| maxLength: 64kb | |||
There was a problem hiding this comment.
maxLength can only be number
| @@ -11548,7 +11563,7 @@ components: | |||
| description: >- | |||
| Whether translator instructions have been captured for the | |||
| file | |||
| type: integer | |||
There was a problem hiding this comment.
description indicates it is boolean
|
|
|
Very common lint error: schema does not match examples. In many cases, the schema is defined incorrectly. Sometimes arrays are defined where there has to be an object, sometimes, an object is defined in place of an array. |
|
@AlexArcher idk what Edit: |
* JOBSBATCH-213 #251 #252 Partially fixed warnings in the Batch API * JOBSBATCH-213 #251 #252 Fixed warnings in the Batch API --------- Co-authored-by: Loparev <[email protected]>
Tqct 1058 fix issues and TQC warnings
…-documentation-for-creating-sdk
testscript to validate and lint the documentationnpm run testscript will show linting errors.npm run test -- --format=stylishwill produce a more compact output grouped by filePreview link