-
Notifications
You must be signed in to change notification settings - Fork 47
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
Devex 1100 swagger doc housekeeping #2696
Devex 1100 swagger doc housekeeping #2696
Conversation
This reverts commit 354a8b5.
@@ -16,7 +16,8 @@ exports.swaggerLambda = () => { | |||
<script> | |||
SwaggerUIBundle({ | |||
dom_id: '#swagger', | |||
url: '/api/swagger.json' | |||
url: '/api/swagger.json', | |||
supportedSubmitMethods: [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removes the "try it out!" button from the docs.
"CognitoUserPool": [] | ||
} | ||
] | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
relocation. combined two separate "/cases" blocks.
} | ||
}, | ||
"required": ["showCaseNotes", "filters", "formattedCases", "formattedTrialSession", "sort", "userHeading"] | ||
"$ref": "#/definitions/printableWorkingCopyData" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved this schema from here into its own definition at the bottom of the file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comments for explanation.
just cleaning up this swagger doc:
known issues: swagger UI is unable to properly display nested schemas. the schema in the JSON file is correct; the schema on SwaggerUI is not.