deiu / solid-plume

Plume, a Solid blog platform!
https://deiu.github.io/solid-plume
MIT License
101 stars 22 forks source link

Problem with newly forked rdflib.js #7

Open happybeing opened 6 years ago

happybeing commented 6 years ago

I've forked rdflib.js and plume to create a PoC Solid app for SAFE Network using Plume, but I'm hitting a problem, probably because Plume needs updating to work with the newest rdflib.js.

I've disabled Plume's authentication (working already logged into a local server), so there's a possibility that this or something else is the cause but I think it is most likely my switching to a newer rdflib.js

Anyway, in Chromium browser, when I click Publish on my first blog post rdflib.js throws an error. Console shows:

Uncaught Error: Missing IRI for NamedNode
    at new t (named-node.js:23)
    at Object.u (data-factory.js:43)
    at savePost (app.js:632)
    at Object.publishPost (app.js:625)
    at HTMLButtonElement.onclick ((index):1)

Links to source: at new t (named-node.js:23) at Object.u (data-factory.js:43) at savePost (app.js:632)

happybeing commented 6 years ago

On second thoughts - the error is different with the latest rdflib.js but I was getting an error at the same point (indexOf on undefined I think) when using plume unmodified. So the fault is probably how I'm trying to use plume, not necessarily a problem with rdflib.js compat.

Will leave this open for now - it would still help to have a version of plume that is up to date with rdflib.js.

happybeing commented 6 years ago

I should add - if you were to update, please include the rdflib.min.js.map in your distro! :-)