Skip to content
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

updater.update() fails on insert with error but still can't figure out what editing protocol it supports #637

Closed
bourgeoa opened this issue Feb 18, 2024 · 0 comments · Fixed by #638

Comments

@bourgeoa
Copy link
Contributor

PR modified removeDocument() to also remove metadata.

The consequence is that for example fetcher.load() with options.clearPreviousData=true removes also metadata.

rdflib.js/src/fetcher.ts

Lines 2005 to 2008 in 6ab4f04

// Before we parse new data clear old but only on 200
if (options.clearPreviousData) {
kb.removeDocument(options.resource)
}

Running updater.update() fails on insert with error but still can't figure out what editing protocol it supports.
This is due to the fact that the current response is stored before being deleted just after.

@bourgeoa bourgeoa linked a pull request Feb 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant