Feature/issue 348 post passport

[briandoconnor]

See Issue #348

[dglazer]

OpenAPI question -- do we need to duplicate the content of the POST and GET endpoints? It would be a lot easier to read (and to maintain) if we could share the duplicated part. The current docs show two identically named methods, which is confusing.

[briandoconnor]

Bigger issue @dglazer is this is swagger 2.0... not OpenAPI which is what we should move towards as a project. I'd prefer to not invest a lot of time figuring out how to do this in the old swagger language and focus on cleaning it up in the move to OpenAPI 3

[dglazer]

I agree we should move to OpenAPI 3 asap -- @jb-adams to make sure that's on the list.
I'm not sure that will help with this particular issue, but I agree it's not worth debugging in swagger syntax.

[jb-adams]

Looks like #230 aims to migrate to OpenAPI 3. @jaeddy @uniqueg has there been any movement on this lately? I can take this branch if you have no bandwidth currently

[briandoconnor]

@jb-adams , it would be fantastic to move the OpenAPI 3 work forward and coordinate that with @jaeddy as part of the trans GA4GH effort to have common doc build approaches across our repos.

[uniqueg]

Thanks @jb-adams that would be appreciated. Very little bandwidth here in May and June, unfortunately.

[briandoconnor]

I'm closing this PR in favor of #363 which uses OpenAPI 3