search

linkeddata / rdflib.js

Linked Data API for JavaScript
http://linkeddata.github.io/rdflib.js/doc/
Other
565 stars 143 forks source link

confirm that namespaces link is now correct #481

Closed TallTed closed 3 years ago

TallTed commented 3 years ago

Re: Line 95 of webapp-intro.html

In #321, on 2019-07-21, I was told by @TylerRick that https://github.com/solid/solid-ui/blob/master/src/ns.js should be replaced by https://github.com/solid/solid-namespace/blob/master/index.js. Today, #321 was merged by @jeff-zucker, and I was again asked to make this URL change (which couldn't be done within #321, due to inadvertent branch deletion) with a new PR.

However, while #321 languished, @janschill replaced https://github.com/solid/solid-ui/blob/master/src/ns.js with https://github.com/solid/solid-namespace, which is there now.

I don't know which of https://github.com/solid/solid-namespace or https://github.com/solid/solid-namespace/blob/master/index.js is correct and should persist. I'm sure someone reading this will know, and will either close this with "it's correct now" or tell me to make (or will make their own) PR to change it to the other.

jeff-zucker commented 3 years ago

Thanks @TallTed - the one Jan used is probably better, so nothing needs to be done. The only reason I asked you to make the PR on the other one instead of doing it myself is that I'm pretty sure that if I made the PR, I couldn't also merge the PR, and thought it would be faster for you to do it and me to merge it than for me to do it and wait for someone else to merge it.