Skip to content

Conversation

@pvgenuchten
Copy link
Contributor

@pvgenuchten pvgenuchten commented Oct 28, 2022

Overview

This is a suggestion on how contact details can be captured on pycsw database, so they can be used on ogc-api-records.

It maps also topiccategory and otherconstraints to geojsonrecord properties (license and theme)

Related Issue / Discussion

#806

Additional Information

a new field 'contacts' has been added to the pycsw database

Contributions and Licensing

  • I'd like to contribute this feature to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.

@pvgenuchten pvgenuchten force-pushed the extend_recordgeojson branch 5 times, most recently from a3cfd50 to 5146413 Compare October 31, 2022 12:05
@kalxas
Copy link
Member

kalxas commented Nov 2, 2022

Thank you @pvgenuchten for the PR.
I would prefer if we avoid adding jsons as a new dependency.

@kalxas kalxas requested a review from tomkralidis November 2, 2022 17:24
Copy link
Member

@tomkralidis tomkralidis left a comment

Choose a reason for hiding this comment

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

Great work @pvgenuchten. See change requests.

@kalxas
Copy link
Member

kalxas commented Nov 24, 2022

We need to test if the new field is populated properly when importing ISO-19115 XML

@pvgenuchten
Copy link
Contributor Author

We need to test if the new field is populated properly when importing ISO-19115 XML

I tested for iso, but not for others like dc

@pvgenuchten pvgenuchten force-pushed the extend_recordgeojson branch 2 times, most recently from cc30d89 to e59c57f Compare November 28, 2022 08:04
@tomkralidis tomkralidis merged commit 2b94076 into geopython:master Dec 6, 2022
@pvgenuchten
Copy link
Contributor Author

nice, this got merged, thanx for reviewing!

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