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
added starter JSON schemas in appendix #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, this must be a ton of work!
"type": "array", | ||
"items": { | ||
"type": "string", | ||
"pattern": "^[a-z]{2,}-[A-Z]{2,}$"/* will need a better pattern for language tags*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I think we'll have to punt this one to the application.
} | ||
} | ||
}, | ||
"subject_id_types": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would benefit from a common "include file".
}, | ||
"updated_at": { | ||
"type": "string", | ||
"pattern": "([0-9]{4})-([0-9]{2})-([0-9]{2})T([0-9]{2})\\:([0-9]{2})\\:([0-9]{2})" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure this pattern is needed, you should have an ISO8601 parser anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's as much value as defining a string field as a uri
type, since an application would have a URI parser also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but here you're using the schema definition to define a time format, maybe even a new time format.
The schema has changed significantly since this was submitted, closing now and tracking the need for this in #166 |
This adds a proposed JSON Schema to the appendix as a resource for developers. This schema currently covers the main request and response messages.