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

Additional formats as discussed in #607 #811

Closed
wants to merge 1 commit into from
Closed

Additional formats as discussed in #607 #811

wants to merge 1 commit into from

Conversation

ralfhandl
Copy link
Contributor

Additional formats as discussed in #607

@ralfhandl ralfhandl changed the title https://github.com/OAI/OpenAPI-Specification/issues/607 Additional formats as discussed in #607 Oct 6, 2016
@darrelmiller
Copy link
Member

I think these additional formats would be valuable additions to a "format registry" that indicates commonly used formats. I'm not convinced that all of these formats should be defined in the specification. This would require all tooling to support all of the formats and I suspect many would not be.

@ralfhandl
Copy link
Contributor Author

I'm not convinced that all of these formats should be defined in the specification.

Which are the new ones that CAN be defined in the specification?

Given that there's no "format registry" and the Swagger/OpenAPI specification seems to be the only official document that defines formats beyond the rather rudimentary list in http://json-schema.org/latest/json-schema-validation.html#rfc.section.7.3, I'll happily settle for any non-empty subset of the new formats.

@webron webron mentioned this pull request Nov 18, 2016
@webron
Copy link
Member

webron commented Nov 18, 2016

@ralfhandl - thanks for the PR. The @OAI/tdc has discussed the issued and decided that we don't feel the extended format descriptions should be in the spec.

However, there's an understandable need to define such additional formats and we've considered two alternative approaches. These will be handled in #845.

@webron webron closed this Nov 18, 2016
@ralfhandl ralfhandl deleted the issue-607 branch August 22, 2017 09:54
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.

3 participants