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

Add support for batch credential request #18

Merged
merged 9 commits into from
Sep 3, 2024

Conversation

theosirian
Copy link
Collaborator

@theosirian theosirian commented Aug 22, 2024

The changes are not up to date to the current main branch, I'll be updating them today, but I believe it can already be reviewed keeping the potential types/signatures changes in mind.
The branch is now rebased.

@theosirian theosirian force-pushed the feat/updates-for-sprucekit-integration branch from 227f98b to 4ad5446 Compare August 22, 2024 16:02
Copy link
Member

@sbihel sbihel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you change the title of the PR to better reflect the changes in this PR?

Good work -- I'm requesting changes but it's mostly for confirmation that the changes to the credential configuration struct are correct

src/client.rs Show resolved Hide resolved
src/client.rs Outdated Show resolved Hide resolved
src/credential.rs Outdated Show resolved Hide resolved
src/credential_offer.rs Show resolved Hide resolved
src/core/profiles/w3c/ldp.rs Show resolved Hide resolved
src/proof_of_possession.rs Show resolved Hide resolved
@sbihel sbihel changed the title Updates to support SpruceKit integration Add support for batch credential request Aug 29, 2024
@cobward cobward merged commit 020887f into main Sep 3, 2024
1 check passed
@cobward cobward deleted the feat/updates-for-sprucekit-integration branch September 3, 2024 08:36
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.

3 participants