Adds render cross reference in links #6
Conversation
This PR Adds render cross reference in links and fix #2
| @@ -34,6 +34,37 @@ | |||
| } | |||
| } | |||
| } | |||
| }, | |||
| { | |||
| "if": { | |||
There was a problem hiding this comment.
Is this schema saying if stac extensions includes both renders and web-map-links, a render sub-property is required? Why do you think it should be required? Seems like an optional property.
There was a problem hiding this comment.
Also, I may be mis-reading the schema but does the then statement mean that there must be a link object in the links list that has "rel": "render" - in which case does the example above needs updating to include a link with "rel": "render"?
|
@emmanuelmathot this makes sense to me but I wonder if this spec should be in the web-map-links extension rather than here, since the change is in the links object, not the renders object. |
The change specifies an extension to link but specific to render so this should be set here. If the logic discussed here: radiantearth/stac-spec#1284 similar to this proposition changes, we will adapt to the logic but for the moment, we will add the possibility to link a render in a link. |
This PR Adds render cross reference in links and fix #2