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

#31 merge global local schema #54

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

KajKandler
Copy link
Contributor

John,
I had a look at the global/local schema merge issue #31, as I reported.

This change is rather convoluted:

  • I had to write an odd document_loader for jsonld because their signature assumes that all schema is loaded from a URL (it is a beta after all)
  • I had to refactor quite some code, because the injection of schema does not work, after the entire schema is converted to a "@graph", which is fine with me. Let me know if you'd rather not have this and I'll see if/how I can make this less restrictive (there are hints in the jsonld lib, but not sure how it works.

@KajKandler KajKandler force-pushed the #31__merge_global_local_schema branch from 5f00111 to 9911221 Compare March 4, 2024 15:48
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 this pull request may close these issues.

1 participant