search

GLEIF-IT / lei-rdf

RDF LEI Project
MIT License
8 stars 4 forks source link

Issue 205 #206

Closed dallemang closed 3 years ago

dallemang commented 3 years ago

These are all pretty much syntactic things; the encodings of strings with special characters, fixing leading spaces in URIs, then adding in the riot conversion to ttl.

I haven't been able to test the whole thing in the docker context. I need to talk to @bryonjacob about how to create a test environment to do that.

dallemang commented 3 years ago

I've made more changes than you asked for; I noticed that repex was still in RDF, so I figured I should do that one into ttl as well, but that one had a more serious XSLT error in it (issue-209). So I had to fix that before fixing this. I left them both in this one PR.

Just as before, my workflow doesn't give me a way to do an integration test; I have tested the new GLEIF-repex.xsl on a test file, but not the full one, because I don't have the streaming license in my test environment. And I haven't tested the whole thing together, I can only do that once the PR is approved.

dallemang commented 3 years ago

I don't see how I can tag someone for approval again; can you just comment on the new repex stuff? @rivettp