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

feat: adding OGC API Feat. to external link for 19115-3 #256

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

f-necas
Copy link
Collaborator

@f-necas f-necas commented Nov 8, 2023

This PR is intended to add external link to OGC API - Features for the 19115-3 and 19139 iso.

geonetwork/core-geonetwork#7474

@fvanderbiest fvanderbiest added this to the 23.1.0 milestone Nov 8, 2023
@f-necas f-necas merged commit b17e8cb into georchestra-gn4.2.x Nov 28, 2023
1 check passed
@f-necas f-necas deleted the 4-ogcapifeature-19115-3-geor branch November 28, 2023 16:10
"isMultilingual": false
},
"protocol": {
"value": "OGC API - Features",
Copy link
Contributor

Choose a reason for hiding this comment

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

I have thought again about this value here; I think it would at least be better to align with the "official" values that should land in GeoNetwork soon, see geonetwork/core-geonetwork#7468

The value should then be OGC API Features instead of OGC API - Features. Otherwise records in geOrchestra might not be correctly understood once the PR above is merged.

Sorry for not picking that up earlier 😕

Copy link
Member

Choose a reason for hiding this comment

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

Thanks @jahow !
We'll take care of updating this locally.

Copy link
Member

Choose a reason for hiding this comment

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

@f-necas f-necas modified the milestones: 23.1.0, 24.0.0 Dec 19, 2023
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