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 custom offers from the proposal #1426

Conversation

acuderman
Copy link
Contributor

What is this PR for?

Currently, the issuer only has the ability to offer a credential that is the same as the one defined in a credential proposal.
This PR enables the holder and issuer to negotiate about the credential via proposals and offers in order to agree on what credential the holder will receive.

Backwards compatibility:

If the counter-offer is not defined/empty logic stays the same as It was before this change.

@acuderman acuderman force-pushed the feat-IDENT-3499-json-ld-credential-offer-proposal-fix branch from cd45b60 to cf46cb9 Compare September 29, 2021 12:35
@acuderman acuderman changed the title Add support for offers from the proposal Add support for custom offers from the proposal Sep 29, 2021
@codecov-commenter
Copy link

codecov-commenter commented Sep 29, 2021

Codecov Report

Merging #1426 (cd45b60) into main (024c624) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head cd45b60 differs from pull request most recent head cf46cb9. Consider uploading reports for the commit cf46cb9 to get more accurate results

@@            Coverage Diff             @@
##             main    #1426      +/-   ##
==========================================
- Coverage   95.30%   95.30%   -0.01%     
==========================================
  Files         483      483              
  Lines       29237    29236       -1     
==========================================
- Hits        27865    27864       -1     
  Misses       1372     1372              

@andrewwhitehead andrewwhitehead merged commit 9bea1d2 into openwallet-foundation:main Sep 29, 2021
@ctomc ctomc deleted the feat-IDENT-3499-json-ld-credential-offer-proposal-fix branch August 18, 2022 09:53
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.

4 participants