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

Consider support for JSON References #295

Closed
Hypercubed opened this issue Sep 12, 2016 · 6 comments
Closed

Consider support for JSON References #295

Hypercubed opened this issue Sep 12, 2016 · 6 comments
Assignees

Comments

@Hypercubed
Copy link

As discussed in #293 we should consider adding a $schema tag to the JSON Table Schema spec to distinguish between JSON Schema and JSON Table Schemas. Subsequently, I suggest that the Data Package spec support specifying JSON Schemas for arbitrary data. To do this I believe the Data Package spec should include support for JSON References.

@pwalsh
Copy link
Member

pwalsh commented Sep 27, 2016

I think JSON Reference support is useful.

I think we should consider this for a post v1 feature of the specs.

@rufuspollock
Copy link
Contributor

Generally a +1 though may be post v1 as @pwalsh suggests.

@rufuspollock
Copy link
Contributor

@Hypercubed could you give a bit more detail on how you anticipate this working in the spec and being interpreted by tools.

@Hypercubed
Copy link
Author

I think the point is that if Data Package spec is to support JSON Schemas (in addition to JSON Table Schemas) then JSON Pointer support is needed. From draft-wright-json-schema-00:

Any time a subschema is expected, a schema may instead use an object containing a "$ref" property. The value of the $ref is a URI Reference. Resolved against the current URI base, it identifies the URI of a schema to use. All other properties in a "$ref" object MUST be ignored.

@rufuspollock
Copy link
Contributor

rufuspollock commented Dec 3, 2016

@Hypercubed so you imagine in a resource object something like this:

"schema": {
  "$ref": URI reference ...
}

@pwalsh
Copy link
Member

pwalsh commented Feb 5, 2017

We won't merge #337 until we can support all the changes in the core implementations we maintain. So closing this as JSON Pointers are implemented in #337 , as leaving it open creates confusion.

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

No branches or pull requests

3 participants