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

Fix SensorThings URI field retention #1924

Merged
merged 3 commits into from
Feb 5, 2025

Conversation

webb-ben
Copy link
Member

@webb-ben webb-ben commented Feb 5, 2025

Overview

This PR fixes the retention and setting of the uri_field for the Sensorthings Provider. This is to ensure that the uri field is present in the response if configured. This also fixes an issue with STA incorrectly setting the URI field.

Related Issue / discussion

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 webb-ben requested a review from a team February 5, 2025 17:27
@tomkralidis tomkralidis added this to the 0.20.0 milestone Feb 5, 2025
@tomkralidis tomkralidis self-assigned this Feb 5, 2025
@tomkralidis tomkralidis added OGC API - Features OGC API - Features bug Something isn't working labels Feb 5, 2025
@tomkralidis tomkralidis merged commit cd6d88f into geopython:master Feb 5, 2025
3 checks passed
@webb-ben webb-ben deleted the sta-fixes branch February 5, 2025 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working OGC API - Features OGC API - Features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants