opengeospatial / ogcapi-coverages

OGC API - Coverages draft specification
https://ogcapi.ogc.org/coverages
Apache License 2.0
22 stars 13 forks source link

Review OpenAPI schemas and definitions, linking to external Common ones #151

Closed jerstlouis closed 2 years ago

jerstlouis commented 2 years ago
cmheazel commented 2 years ago

Review and updates are complete. There is a reference to confClasses.yaml which does not resolve. Investigating

jerstlouis commented 2 years ago

In 13.2 Media Types there is a 404 link to Coverages schema not on beta, let's please change to point to GitHub for now.

https://docs.opengeospatial.org/DRAFTS/19-087.html#_cis_json_encoding

jerstlouis commented 2 years ago

The commit for this issue broke the OpenAPI definition. We also clarified that we should not be using beta.opengis.net for draft specifications. The schemas here were also much more recent, with a modular reorganization based on the work in OGC API - Tiles that made it easy to generate OpenAPI definitions, so referencing the older OGC API - Common schemas did not work as intended. Suggesting that we partially revert this commit, as done in PR #168.

jerstlouis commented 2 years ago

SWG 2022-07-27: PR #168 fixes the broken API definition. Further synchronization efforts with Common and other API specifications would involve trying to maintain the same directory structures. Schemas could be identical (Unless an API suggests a common backward-compatible extension), and could be automatically regularly compared for differences between the different specs and Common. Other API building blocks such as responses and parameters might slightly differ, as these are an example API definitions and might be tailored for the purpose of illustrating the target specification (e.g., the conformance response would list the relevant conformance classes, the f parameter would list the relevant formats). Cross-SWG discussions could happen in OGC API - Common SWG or another forum to discuss bringing in new building blocks from the different API specs into Common specifications.