Skip to content

Openapi fixes#579

Open
jerstlouis wants to merge 2 commits intoopengeospatial:masterfrom
jerstlouis:openapi-fixes
Open

Openapi fixes#579
jerstlouis wants to merge 2 commits intoopengeospatial:masterfrom
jerstlouis:openapi-fixes

Conversation

@jerstlouis
Copy link
Copy Markdown
Member

This fixes new OpenAPI errors and issue #572.

- bindings: additionalProperties and oneOf at wrong indent level
- rename Prefer: return header and declaring it as a shared component to fix some Swagger Editor errors
- updated bundle
@jerstlouis jerstlouis requested a review from pvretano April 15, 2026 19:19
@jerstlouis
Copy link
Copy Markdown
Member Author

cc. @fmigneault

Location for redirection to an [OGC API landing page](https://schemas.opengis.net/ogcapi/tiles/part1/1.0/openapi/schemas/common-core/landingPage.yaml) (for `response=landingPage`) as described in
[OGC API - Common - Part 1: Core](http://docs.ogc.org/DRAFTS/19-072.html#_landing_page_requirements_class) or
an [OGC API collection description](https://schemas.opengis.net/ogcapi/tiles/part1/1.0/openapi/schemas/common-geodata/collectionInfo.yaml) (for `response=landingPage`) as described in
Location for redirection to an [OGC API collection description](https://schemas.opengis.net/ogcapi/dggs/1.0/openapi/schemas/common-geodata/collectionDesc.yaml) (for `response=collection`) as described in
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Using dggs-based nested collection schema seems rather confusing/misleading.

Copy link
Copy Markdown
Member Author

@jerstlouis jerstlouis Apr 16, 2026

Choose a reason for hiding this comment

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

@fmigneault As I said in the commit comment, it's temporary and should be changed to the Common schema which should be published before Processes V2. The previous version was using Tiles which was an older version. Alternatively we could use the GitHub from Common: https://github.com/opengeospatial/ogcapi-common/blob/master/collections/openapi/schemas/common-geodata/collectionDesc.yaml , or even the one in Processes: https://github.com/opengeospatial/ogcapi-processes/blob/master/openapi/schemas/common-geodata/collectionDesc.yaml . I just tried to keep it as a link to a published schema.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes. I think the GitHub one better communicates that it is temporary. Easier to find afterward also for replacement.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@fmigneault Updated.

- Fixes opengeospatial#572
- Also fixed what should have been response=collection, and updated links to published versions of standard document
- NOTE: The links to the collectionDescription schema and OGC API - Common - Part 2 document should be updated to the published versions,
  which will likely happen before Processes v2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants