-
Notifications
You must be signed in to change notification settings - Fork 102
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
Upgrade to strong-swagger-ui@21.0 (swagger-ui@2.1) #114
Conversation
@slnode test please |
@@ -16,12 +21,27 @@ $(function() { | |||
apiKey: '', | |||
dom_id: 'swagger-ui-container', | |||
supportHeaderParams: true, | |||
supportedSubmitMethods: ['get', 'post', 'put', 'delete'], | |||
supportedSubmitMethods: ['get', 'post', 'put', 'delete', 'patch'], |
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.
What about head
?
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.
AFAICT, the flag "supportedSubmitMethods" controls whether operation's model is editable or readonly.
By default, this array is set to all HTTP verbs:
To be honest, I don't completely understand the purpose of this option. I just applied the value from swagger-ui's index.html. Perhaps we can remove this line completely and use swagger-ui's default instead?
I have removed @raymondfeng LGTY now? |
LGTM |
2d76f3c
to
30017c8
Compare
@slnode test please |
…-2.1 Upgrade to strong-swagger-ui@21.0 (swagger-ui@2.1)
This patch upgrades loopback-explorer to use strong-swagger-ui@21.0, which is based on swagger-ui@2.1 and supports Swagger Spec 2.0.
Notes
GET /models
, the response model is not correctly rendered ATM. I suspect this is caused by Pick up enhancement from shelbys #111 and/or Pick up enhancements from shelbys master #113, I'll be investigating the problem after this PR is landed.Connect to strongloop-internal/scrum-loopback#412
Connect to #54
/to @raymondfeng @STRML please review