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

Expose WMS Layer and WFS FeatureType MetadataURLs #71

Merged
merged 5 commits into from
Oct 28, 2024

Conversation

ejn
Copy link
Contributor

@ejn ejn commented Sep 25, 2024

Applications may want to provide links to available metadata of layers and feature types shown. For this to be possible it is necessary to expose the relevant elements in the API.

Note that some changes to test fixtures were required in order to harmonise the MetadataURL elements included in different WFS versions.

Copy link
Member

@jahow jahow left a comment

Choose a reason for hiding this comment

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

Thank you so much for all these great contributions! I can't see if the tests passed on this one, but the code looks good so I'll approve it.

ejn added 5 commits October 28, 2024 09:47
Map applications may wish to provide links to the metadata declared for
WMS layers, so it is necessary to expose this information, encoded as
one or more MetadataURL elements in the WMS layer, in the API.
Use a common structure for metadata from all capabilities document types
Applications may wish to provide links to metadata for FeatureTypes
shown.
Ensures that MetadataURL is comparable across versions
@ejn ejn force-pushed the feature-metadata-url branch from cf1fc6a to a483d78 Compare October 28, 2024 08:48
@ejn
Copy link
Contributor Author

ejn commented Oct 28, 2024

@jahow I've rebased this one on to the current state of main to deal with the conflicts

@jahow jahow merged commit 3e2d3cc into camptocamp:main Oct 28, 2024
1 check passed
@jahow
Copy link
Member

jahow commented Oct 28, 2024

All good!

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.

2 participants