-
Notifications
You must be signed in to change notification settings - Fork 18
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
$rdf is not defined #225
Comments
Hmm, are you sure you aren't doing anything else other than what you show there? I copied it locally and changed only the target and jwt and got no errors and the stuff cloned correctly. AFAIK the only reason solid-node-client would need $rdflib is the stuff mentioned in the README about patch of local files. |
Sorry, I didn't include the call stack.
The problem seems to be here https://github.com/solid/solid-rest/blob/main/core/src/handleResponse.js#L116. Any ideas? I don't think I'm doing anything else locally. The project uses typescript, but I don't think that should affect anything. |
Oh, darn, yes. I added ability to pass JSON-LD in the accept header and that uses $rdf. It would only get triggered when passed an explicity accept header which is why it failed for you and not me. I'll fix trap it and document it. Thanks! |
Cool thanks! |
Hey @jeff-zucker do you know when this will be fixed? |
I will do it today.
…On Mon, Feb 21, 2022 at 10:51 PM jaxoncreed ***@***.***> wrote:
Hey @jeff-zucker <https://github.com/jeff-zucker> do you know when this
will be fixed?
—
Reply to this email directly, view it on GitHub
<#225 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKVJCJFPYCTATNWZLN4ESHLU4MXAZANCNFSM5OSWXI6A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Version 2.1.1 of solid-node-client should now include version 2.1.8 of @solid-rest/file which should now include version 2.14 of @solid-rest/core which should solve this problem in solid-file-client. |
Awesome! Thanks |
Okay, I'll close out this issue, but I've encountered another problem in this issue: solid-contrib/solid-rest#76 |
Hey, sorry for the delay in testing. I'm still experiencing the problem. To recreate it, go to this repo: https://github.com/o-development/clone-to-solid-pod and follow the instructions labeled "Running the program without the github hook" in the readme. You should see the message |
Fixed in o-development/clone-to-solid-pod#1 |
Hey Jeff,
First time using solid-file-client. I had some trouble with the following error:
ReferenceError: $rdf is not defined
. It seems that something wants rdflib, but it can't get it. Here's my code:[email protected] and [email protected]
The text was updated successfully, but these errors were encountered: