This repository was archived by the owner on Jul 10, 2025. It is now read-only.
feat: provide context loader for json-ld methods - #8
Closed
GlugovGrGlib wants to merge 1 commit into
Closed
Conversation
Copy link
Copy Markdown
Sorry, something went wrong.
This commit solves an error[E0061] when building against latest versions of the didkit and ssi libraries
Copy link
Copy Markdown
Member
|
Thanks for the PR. Just to confirm, we are talking about this context file https://github.com/spruceid/ssi/blob/main/contexts/jff-vc-edu-plugfest-2-context.json ? Yes, I'll need to publish new versions of ssi and didkit -- is it ok if it is done next week? |
Sorry, something went wrong.
Copy link
Copy Markdown
Author
Yes, this context is required for our use case.
Sure, this will be great, thank you! |
Sorry, something went wrong.
sbihel
added a commit
that referenced
this pull request
Mar 9, 2023
sbihel
added a commit
that referenced
this pull request
Mar 9, 2023
Also remove Linux architectures without ring support Close #8
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds support for ContextLoader feature which introduced in: spruceid/ssi#432 and spruceid/didkit#298.
We, at openedx.org, want to use didkit-python to issue OpenBadgesV3 with StatusList2021 support. Although this kind of functionality was already added to ssi, changes in this PR must be applied to fix an error[E0061] when building against the latest versions of didkit and ssi.
This PR was tested locally by building didkit-python using main branch for didkit and ssi.
Do I understand correctly, to release a new didkit-python version with these changes, it's necessary to republish didkit on crates.io, and then update the version and the list of features in Cargo.toml, as currently enabled features were already removed from didkit?