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.

Sign up for GitHub

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

Jump to bottom

Issue Credential v1 and Present Proof v1 #1056

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

Issue Credential v1 and Present Proof v1 #1056

wants to merge 3 commits into from

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Sep 24, 2024 •
edited
Loading

Opening early.

This PR moves ICv1 and PPv1 to a plugin.

This description will be updated with notes on any sharp edges encountered.

Plugin-ification

ICv1

  • Credential revocation state used to get updated by the revocation manager by using the exchange record directly. Changed to trigger an event instead.

PPv1

  • OOB depends on recalling pres ex records to populate attachments.
  • Pres Exch handler depends on recalling pres ex records to retrieve proof request.

@dbluhm
feat: copy over icv1 and ppv1
62e2313 
Signed-off-by: Daniel Bluhm <[email protected]>
@dbluhm dbluhm changed the title feat: copy over icv1 and ppv1 Issue Credential v1 and Present Proof v1 Sep 24, 2024
@esune esune mentioned this pull request Oct 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dbluhm