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

Remove the path to items and its definition #110

Closed
joanma747 opened this issue Mar 15, 2020 · 6 comments
Closed

Remove the path to items and its definition #110

joanma747 opened this issue Mar 15, 2020 · 6 comments
Labels
Collections Applicable to Collections (consider to use Part 2 instead) Guide help wanted Extra attention is needed Progress: resolution agreed

Comments

@joanma747
Copy link
Contributor

This issue has been created as part of the public review and is based on document 19-072. See in particular section 9.2.3.

Not all geospatial data is structured in "items". "/collections/{collectionsId}/items" is defined by "OGC API features" because they need it. OGC API coverages are planning to use /collections/{collectionsId} but they will never use "items". EDR are also using collections but they are not using items so far. The only other proposed OGC API candidate that I know is that proposes the use of "items" is OGC API records and it is still under discussion if this is convenient.

(the discussion on the convenience of the word "collections" versus "geodata" or similar is not part of the scope of this CR).

OGC API Common should NOT define "items". Please remove.

In addition also remove any reference to itemsType

@cmheazel cmheazel added the Resources of Collections type Issues related to the /collections path label Mar 25, 2020
@jeffharrison
Copy link

+1

@cmheazel cmheazel added the Collections Applicable to Collections (consider to use Part 2 instead) label May 11, 2020
@dblodgett-usgs
Copy link

This needs a motion at the next API Common meeting. Highly related to #107.

Motion: Remove items and itemType from OGC-API Common Part 2.

@cmheazel cmheazel self-assigned this Jul 27, 2020
@cmheazel
Copy link
Contributor

API-Common Part 2 has been rewritten so that /items is optional. The decision of whether or not to include /items will lie with the SWGs developing API Standards which build on Part 2.
This is an uncomfortable solution. /items is too useful to not make common, not universal so it cannot be mandatory, and too small to make its' own conformance class.

@bradh
Copy link
Contributor

bradh commented Aug 14, 2020

Maybe Common should provide a set of guidance / recommendations. Perhaps a pattern approach of describing the concept, showing an implementation, and noting benefits and (potential) downsides.

@cmheazel
Copy link
Contributor

@bradh This material should be captured in the Users Guide, not the Standard. If someone were to contribute content for the Users Guide, then we can point to it from the Standard.

@cmheazel cmheazel added Guide and removed Collections Applicable to Collections (consider to use Part 2 instead) labels Aug 17, 2020
@cmheazel cmheazel added Collections Applicable to Collections (consider to use Part 2 instead) help wanted Extra attention is needed and removed Resources of Collections type Issues related to the /collections path labels Sep 11, 2020
@cmheazel cmheazel removed their assignment Sep 11, 2020
@cmheazel
Copy link
Contributor

#221 removes the /items endpoint. That makes this issue OBE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Collections Applicable to Collections (consider to use Part 2 instead) Guide help wanted Extra attention is needed Progress: resolution agreed
Projects
Development

No branches or pull requests

5 participants