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

INSPIRE ATOM - retrieve the metadata ATOM feed using only the resource identifier #7305

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

josegar74
Copy link
Member

Support series metadata linked in INSPIRE ATOM services in the ATOM dataset describe service, otherwise a 404 error is returned in requests like this for series metadata.

https://localhostgeonetwork/srv/atom/describe/dataset?spatial_dataset_identifier_code=https://doi.org/10.11/3028ccf4-ff64-42ed-98f5-fea1b32da62a&language=de

@josegar74
Copy link
Member Author

@fxprunayre I'm not sure if we really need to filter by the resource type, probably would be better just to filter by the identifier code / codename to retrieve the feeds linked in the service metadata.

What do you think?

@fxprunayre
Copy link
Member

@fxprunayre I'm not sure if we really need to filter by the resource type, probably would be better just to filter by the identifier code / codename to retrieve the feeds linked in the service metadata.

Indeed the code is supposed to be precise enough. Not sure there is any INSPIRE restriction about the type. Editors have to take care to link to proper records.

@josegar74 josegar74 changed the title INSPIRE ATOM - support metadata series INSPIRE ATOM - retrieve the metadata ATOM feed using only the resource identifier Sep 1, 2023
@josegar74
Copy link
Member Author

Update the code to retrieve the metadata ATOM feed using only the resource identifier, not the resource type.

The user should be responsible to link the proper records.

Sonarlint improvements are included also.

@josegar74 josegar74 merged commit 79865d1 into geonetwork:main Sep 7, 2023
6 checks passed
josegar74 added a commit that referenced this pull request Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants