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

Update GC_Open_Licenses.rdf #303

Closed
wants to merge 1 commit into from
Closed

Conversation

CarHenkel
Copy link

Change on two lines HTTP -> HTTPS. ECCC DMS Management wants to eliminate all HTTP in XML. @josegar74

Change on two lines HTTP -> HTTPS.  ECCC DMS Management wants to eliminate all HTTP in XML.
@bo-lu
Copy link
Member

bo-lu commented Feb 15, 2023

@CarHenkel This may break validation which occurs on the license value. It will also coordination to the 3.6.x branch and upgrade existing records in the existing catalogues (FGP/DFO/ECCC). Let's follow up with Jose so all the needed changes are coordinated.

@bo-lu
Copy link
Member

bo-lu commented Feb 15, 2023

I've outlined some tasks in #304 which need to be completed first so existing flows are preserved.

@CarHenkel
Copy link
Author

Thank you so much! Let's chat about this in the technical community meeting. We're hoping to tackle some RDF issues next fiscal. (Never done a pull request before. Hope I went about it the correct way :) Jérôme is tackling a bulk update of existing records

I've implemented the RDF on our staging server, and it does indeed not internally validate until the list item is reselected by the user. I presume it would then fail FGP validation.
https://iso19139echnap.geocat.live/geonetwork/srv/eng/catalog.search#/home

image

@CarHenkel CarHenkel closed this Feb 15, 2023
@bo-lu
Copy link
Member

bo-lu commented Feb 15, 2023

Definitely, it has been on our issues tracker since 2020! It would be nice to have it fixed this FY. I will do what I can on our side. Cheers!

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