-
Notifications
You must be signed in to change notification settings - Fork 9
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
XML produced by org-id.guide appears to be invalid against IATI schema #204
Comments
Thanks for looking into this. My question here is whether we should edit this in the data, e.g. https://github.com/org-id/register/blob/master/lists/pk/pk-vswa.json#L7 , or change our XML code to handle the data. I think changing the data is probably better, as commas in a url field seems like something that might trip other people up. @timgdavies What do you think? |
Agreed that we should change the source. I'll look into this shortly and will revert. |
Great. Once this is sorted, I’ll update the PR to IATI. |
I've updated org-id.guide sources so it should be possible to update the PR from the live source. |
Thanks @timgdavies - lgtm! |
This XML:
http://org-id.guide/download.xml
…appears to be invalid against the IATI schema. Here’s a travis build where it fails to validate:
https://travis-ci.org/IATI/IATI-Codelists-NonEmbedded/builds/364483003
The issue appears to be instances where multiple comma-separated URLs are provided in the URL field. Like this one:
https://github.com/andylolz/IATI-Codelists-NonEmbedded/blob/915a02da95a876e6f852cb4ac5eb2321eb878f60/xml/OrganisationRegistrationAgency.xml#L2440
I suppose it would be preferable to pick a single URL, and perhaps add any other URLs to the description.
I ran into this problem whilst trying to update the relevant IATI codelist from org-id.guide (IATI/IATI-Codelists-NonEmbedded#247).
The text was updated successfully, but these errors were encountered: