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

fix(anoncreds): wrong key name for predicates in proof object #1517

Conversation

auer-martin
Copy link
Contributor

No description provided.

@auer-martin auer-martin requested a review from a team as a code owner July 21, 2023 14:21
@TimoGlastra TimoGlastra enabled auto-merge (squash) July 21, 2023 14:23
@TimoGlastra
Copy link
Contributor

There are some errors with the type checking

auto-merge was automatically disabled July 26, 2023 14:15

Head branch was pushed to by a user without write access

ryjones and others added 3 commits July 26, 2023 16:16
Co-authored-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Signed-off-by: Ry Jones <ry@linux.com>
Signed-off-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Signed-off-by: Martin Auer <martin.auer97@gmail.com>
Signed-off-by: Martin Auer <martin.auer97@gmail.com>
Signed-off-by: Martin Auer <martin.auer97@gmail.com>
@@ -19,6 +19,7 @@ jobs:
with:
# pulls all commits (needed for lerna to correctly version)
fetch-depth: 0
persist-credentials: false
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this change should be reverted from this PR

Copy link
Contributor Author

@auer-martin auer-martin Jul 26, 2023

Choose a reason for hiding this comment

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

i removed it sorry

Signed-off-by: Martin Auer <martin.auer97@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #1517 (fa8bdca) into main (6795426) will increase coverage by 23.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #1517       +/-   ##
===========================================
+ Coverage   62.83%   85.84%   +23.00%     
===========================================
  Files         778      948      +170     
  Lines       17969    22627     +4658     
  Branches     3082     3957      +875     
===========================================
+ Hits        11291    19424     +8133     
+ Misses       6138     3021     -3117     
+ Partials      540      182      -358     
Files Changed Coverage Δ
packages/anoncreds/src/models/exchange.ts 100.00% <ø> (ø)
...sdk/src/anoncreds/services/IndySdkHolderService.ts 8.02% <0.00%> (ø)
...k/src/anoncreds/services/IndySdkVerifierService.ts 23.52% <ø> (ø)

... and 472 files with indirect coverage changes

@TimoGlastra TimoGlastra merged commit d895c78 into openwallet-foundation:main Jul 26, 2023
7 checks passed
@auer-martin auer-martin deleted the fix/proof-predicate-key-name branch December 4, 2023 13:52
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.

5 participants