Skip to content
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

initial attempt at an Open API description of TAP #2

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pdowler
Copy link
Member

@pdowler pdowler commented Sep 18, 2024

this is TAP-1.1 with parameters
the UWS portion is incomplete but enough to see how to ref the VOSI portion is incomplete but includes some of the user table stuff that is intended for TAP-next

the commented out URLs in openapi.yaml work if the yaml is located at a reachable URL (currently not)

this is TAP-1.1 with parameters
the UWS portion is incomplete but enough to see how to ref
the VOSI portion is incomplete but includes some of the user table stuff that is intended for TAP-next

the commented out URLs in openapi.yaml work if the yaml is located at a
reachable URL (currently not)
@pdowler pdowler marked this pull request as draft September 18, 2024 19:14
@pdowler
Copy link
Member Author

pdowler commented Sep 18, 2024

tap-params.yaml - I was trying to make a single "set of parameters" but could not find a valid way to $ref that in the 3 places it is needed, but maybe the individual parameter definitions are better (more reusable) anyway. I kept this file for now but it is not used.

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

Successfully merging this pull request may close these issues.

1 participant