You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Most changes are on server.js but also added in swaggerui.
I set the model definition on GET /books/ to return an Object instead of an Array which is a common practice to support additional information about the result set.
Is it OK if we change the responses for the listing to be an object with an element for "results" or "books"?
While looking over it, I think if someone is first starting out seeing all this information is overwhelming!
Just recapping what @eerwitt and I talked about earlier. This is a really awesome start for getting us up and running with Swagger, but we won't finish implementing it for week 3, since the API is in a working state for our lessons, and we should focus on lesson planning instead.
It can be an ongoing effort to implement swagger, so we'll leave this issue and the branch open for now.
Better documentation with http://swagger.io
The text was updated successfully, but these errors were encountered: