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

Draft proposal for a collection summary #287

Closed
wants to merge 3 commits into from

Conversation

tschaub
Copy link
Contributor

@tschaub tschaub commented Nov 6, 2019

This is a draft proposal for a "summary" resource that would allow clients to get additional information about items in a collection. I've provided a bit of motivation in the proposal doc itself. This is a very "drafty" draft - plenty of room for discussion on alternative ways to get this done.

I didn't see the Alternative Schema until after creating this PR (and then reshaped this to use the same proposal template). Interesting that that tries to address some of the same concerns. Good evidence that there is motivation.

@tschaub tschaub force-pushed the summary branch 2 times, most recently from d6822fa to 66c3692 Compare November 6, 2019 20:40
@tschaub tschaub changed the title Example summary resource for discussion Draft proposal for a collection summary Nov 6, 2019
extensions/summary/summary.md Outdated Show resolved Hide resolved
extensions/summary/summary.md Outdated Show resolved Hide resolved
extensions/summary/summary.md Outdated Show resolved Hide resolved
extensions/summary/summary.md Outdated Show resolved Hide resolved
extensions/summary/summary.md Outdated Show resolved Hide resolved
### A new resource per extension

We considered the alternative that each extension would add its own resource at a new path. For example, the filter extension might add a `/collections/buildings/queryables` resource. Then the search extension might add a `/collections/buildings/sortables` resource. And then the editing extension might add a `/collections/buildings/writeables` resource. This began to feel unscalable and awkward for clients who want to get a complete picture of how to work with a collection.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and those various capabilities would still have a need to point to the schema, so a lot of redundancy

proposals/002_Summary.md Outdated Show resolved Hide resolved
}
}

Even if this were possible, our concern was that generic schema parsers would not expose these additional properties to a client.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is possible. JSON schema allows for custom keywords and many validators allow to implement custom logic for this.


This proposal doesn't introduce any breaking changes.

## Alternatives considered
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you had a look at the STAC Collection summaries? Even if you not consider it, it would be good if names would not conflict too much...

@cportele cportele added this to In progress in Part 5: Schemas via automation Mar 22, 2021
@cportele cportele moved this from In progress to To do in Part 5: Schemas Mar 22, 2021
@cportele
Copy link
Member

Code Sprint 2024-02-27: A draft of "Part 5: Schemas" exists and addresses the need for feature schemas to describe the feature properties ("returnables"), the editable properties ("receivables"), the properties that can be used in filters ("queryables") and the properties that can be used to sort features ("sortables"). The part is expected to be submitted for OGC Architecture Board Review and then Public Comment soon. We are, therefore, closing this draft PR and invite comments on the draft of Part 5.

@cportele cportele closed this Feb 27, 2024
Part 5: Schemas automation moved this from To do to Done Feb 27, 2024
@tschaub tschaub deleted the summary branch February 28, 2024 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants