grape-swagger icon indicating copy to clipboard operation
grape-swagger copied to clipboard

Swagger 2.0 Support

Open wedtm opened this issue 10 years ago • 38 comments

Is this possible yet?

wedtm avatar Feb 24 '15 08:02 wedtm

really need 2.0 support, swagger 2.0's authorization supports oauth2 4 flows, but only 2 flows for version 1.2.

calfzhou avatar Mar 04 '15 10:03 calfzhou

I'd really like to see https://github.com/tim-vandecasteele/grape-swagger/issues/137, in which we rewrite grape-swagger using a proper swagger DSL library instead of keeping at hacking on stuff. That brings us to Swagger 2.0. And anyone who does it will be my personal hero.

dblock avatar Mar 04 '15 11:03 dblock

:+1:

tbeauvais avatar Apr 15 '15 16:04 tbeauvais

:+1:

danielkummer avatar Apr 24 '15 06:04 danielkummer

Any news about that?

:+1: :+1: :+1:

florrain avatar Jul 01 '15 23:07 florrain

Has anyone thought about this at all yet? Is there a path going forward that we can get started and maybe have multiple people work on it at once?

yairgo avatar Jul 09 '15 17:07 yairgo

Doesn't look like anyone has done anything about this, the first one wins!

dblock avatar Jul 10 '15 19:07 dblock

How about this https://github.com/Gild/ruby-swagger

birbird avatar Jul 25 '15 15:07 birbird

As I understand it's not generating documentation on the fly. But it has support for grape: https://github.com/Gild/ruby-swagger/tree/master/lib/ruby-swagger/grape

dmitry avatar Jul 25 '15 15:07 dmitry

@dmitry correct, there is a rake task - extracting the data from grape into a YML format - this allows tech editors to write all the doc outside the annotations. The documentation can be compiled into its final swagger.json form - this allow us to host the documentation on CDN rather than from the API server.

openmosix avatar Sep 01 '15 13:09 openmosix

:+1:

aceunreal avatar Sep 17 '15 01:09 aceunreal

:+1:

korstiaan avatar Sep 28 '15 13:09 korstiaan

:+1:

wagner avatar Oct 08 '15 00:10 wagner

would be awesome.

Micka33 avatar Nov 17 '15 21:11 Micka33

:+1:

johnhamelink avatar Nov 24 '15 17:11 johnhamelink

/paging @oliverzeyen

rngtng avatar Dec 02 '15 09:12 rngtng

Is the only current strategy to get 2.0 Swagger compliance still https://github.com/Gild/ruby-swagger? Are there any plans to add 2.0 support to this gem?

herberzt avatar Dec 28 '15 21:12 herberzt

Nobody is actively working on this @herberzt. Maybe you will?

dblock avatar Dec 29 '15 01:12 dblock

Can one of the original maintainers outline how someone would go about this task?

Right now, for someone new using this project, it's not necessarily obvious how to go about starting to do this.

@dblock I'm not suggesting you take this task up but it's currently a bit daunting for someone that hasn't maintained this repo before to take up and it's definitely something I'd like to see happen.

edbo avatar Jan 11 '16 11:01 edbo

I am here to help. What I need is a passing build in a pull request, and I can definitely carry it forward after it - https://github.com/ruby-grape/grape-swagger/pull/311 was very promising.

dblock avatar Jan 11 '16 11:01 dblock

https://github.com/ruby-grape/grape-swagger/pull/311 has been merged onto a branch called swagger-2.0. Looking forward to more PRs to move that forward!

dblock avatar Jan 22 '16 15:01 dblock

where do things stand wtih the 2.0 fork and getting it merged in: https://github.com/LeFnord/grape-swagger ?

We are trying it out, but it's unclear where things stand.

pswenson avatar Feb 24 '16 23:02 pswenson

:+1:

XBeg9 avatar Feb 29 '16 17:02 XBeg9

So we have a 2.0 branch. Who wants to take ownership, get that merged onto master, released, etc.? What remains to get done? @LeFnord maybe?

dblock avatar Mar 09 '16 16:03 dblock

Hi guys … I would prefer to create a new grape-swagger-2 repo under the ruby-grape project, so much more people can contribute to it and it feels more on the right place. An side effect of this would be, that we can start a discussion on decisions, cause I don't want to decide all alone, think this would not be a good idea :wink: … there is something to decide, e.g. how to handle grape specific behavior → exactly_one_of, mutually_exclusive to name only some …

LeFnord avatar Mar 09 '16 18:03 LeFnord

I am not in love with the idea of two gems for different versions of the spec. I think people just want to upgrade to the swagger 2.0 spec soon and call it a day, do you see a reason one wouldn't want to?

dblock avatar Mar 10 '16 12:03 dblock

@dblock I hope and I wish … then it would be best as it is for the moment, or? or did you prefer to use my fork for that, this is also ok for me …

LeFnord avatar Mar 10 '16 13:03 LeFnord

I'd like to do the following:

  1. Release 0.10.5.
  2. Branch 0.10.5 into 0.10.x, continue releasing bug fixes only in a 0.10.x series compatible with 1.5
  3. Merge swagger-2.0 onto master, release 0.11.0 with only Swagger 2.0 support.
  4. Move forward.

Thoughts?

If you agree with this @LeFnord I want you to take care of (1), (2), (3) and (4). Aka all of it. I've not been actively developing this library I have been just cheerleading the effort here and would love for more people to take a very active role. Who can be better than the person who wrote the majority of 2.0 support for doing that? :)

dblock avatar Mar 10 '16 21:03 dblock

Hi @dblock mmh ok, but for (1)..(3) I need your advise, cause I thing I haven't the rights to do so, and yeap it also would be the first time :wink:

point (3) would prefer to start with tag 0.20.x, so the other branch has enough version space for changes, and it indicates a little more breaking changes (ok, doing it right SemVer, we have to use 1.0.0) point (4) definitely

LeFnord avatar Mar 11 '16 16:03 LeFnord

@LeFnord I invited you the swagger-maintainers org. I am happy to help with whatever you need.

You should PR yourself under grape-swagger in https://github.com/ruby-grape/ruby-grape.github.io/blob/da76718ff723d8e6e6a3f45972045ce133220842/team/index.md.

Drop me an email at dblock[at]dblock[dot]org with your Rubygems e-mail address for access to that, since you're making the next release.

We should introduce proper semantic versioning, whatever that means, trust your judgement on which version this should be. Also cc: @tim-vandecasteele and @CraigCottingham wrt ^^^ who are the other two maintainers on this project.

Finally, thank you for spearheading this.

dblock avatar Mar 12 '16 14:03 dblock