Skip to content

Conversation

dblodgett-usgs
Copy link
Contributor

@dblodgett-usgs dblodgett-usgs commented Sep 10, 2025

Overview

Notice that you can query for a feature by its id but the attribute you queried for isn't in the response anymore.

https://labs-beta.waterdata.usgs.gov/api/fabric/pygeoapi/collections/nhdplusv2-huc12/items?huc12=170101010306

Or in terms of the pygeoapi demo:

https://demo.pygeoapi.io/master/collections/dutch_windmills/items?gml_id=Molens.1

Notice that the query is for an id that doesn't get returned in the output.

Related Issue / discussion

#2103

Additional information

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@webb-ben
Copy link
Member

Of note, the id_field column is included in the queryables and schema

@webb-ben webb-ben requested a review from a team September 10, 2025 20:01
@tomkralidis tomkralidis self-requested a review September 10, 2025 22:04
@webb-ben webb-ben changed the title don't remove id property Do not remove id_field from properties Sep 10, 2025
@webb-ben
Copy link
Member

webb-ben commented Sep 11, 2025

Related issue #2103

@tomkralidis tomkralidis added the OGC API - Features OGC API - Features label Sep 14, 2025
@tomkralidis tomkralidis added this to the 0.22.0 milestone Sep 14, 2025
@tomkralidis tomkralidis merged commit 219e4cc into geopython:master Sep 14, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OGC API - Features OGC API - Features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants