Skip to content
This repository has been archived by the owner on Aug 4, 2023. It is now read-only.

Example doesn't appear to work with the docs. #49

Closed
avishaan opened this issue Oct 18, 2014 · 11 comments
Closed

Example doesn't appear to work with the docs. #49

avishaan opened this issue Oct 18, 2014 · 11 comments

Comments

@avishaan
Copy link

I am referencing release v0.6.5. This is what I get when I try to look at the docs.

image

@avishaan
Copy link
Author

This might also be of interest.
screen region 2014-10-17 at 17 43 58

@fehguy
Copy link

fehguy commented Oct 18, 2014

Are you building a swagger 1.2 server? It looks like you're doing 2.0, but that error indicates that your swagger ui is too old to support the 2.0 spec

@avishaan
Copy link
Author

Was using the example for 2.0. This occurred when I tried to use the example from the current version build
Are you able to run the current example for v2 swagger for v0.6.5 swagger-tools.

Sent from my iPhone

On Oct 17, 2014, at 5:57 PM, Tony Tam notifications@github.com wrote:

Are you building a swagger 1.2 server? It looks like you're doing 2.0, but that error indicates that your swagger ui is too old to support the 2.0 spec


Reply to this email directly or view it on GitHub.

@avishaan
Copy link
Author

Confirm it is working for swagger-tools 0.5.7

@avishaan
Copy link
Author

Not working in 0.6.1
screen region 2014-10-17 at 18 43 26

@avishaan
Copy link
Author

0.6.0 is working.
image

But this spec wants swagger version as a integer instead of a string when the spec actually calls for a string.

@whitlockjc
Copy link
Member

0.6.1 was where we updated swagger-tools for the latest 2.0 schema. Anything older is not guaranteed to work. I also updated the Quick Start and example to reflect within the past day. I just tested this today so it should work.

@avishaan
Copy link
Author

So I should be able to run v0.6.5 example in the v2.0 folder successfully?

@whitlockjc
Copy link
Member

Correct. If not, something wasn't updated for upstream schema changes and that would need to be fixed. Let me know.

@whitlockjc
Copy link
Member

Wait, I see you're saying it's not working with 0.6.5. I'll look into it when I get home.

@whitlockjc
Copy link
Member

swagger-tools@0.6.6 has been released to fix this. I've also created Issue #50 to create a better test suite for the swaggerUi middleware.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants