Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

fix: support of vp referencing in descriptor_map's path. #3417

Merged

Conversation

vkubiv
Copy link
Contributor

@vkubiv vkubiv commented Oct 25, 2022

Title:
Support of vp referencing in descriptor_map's path.

Summary:

Previous code expected that all items referenced in descriptor_map's path should be valid VC.
Now, this is required only for the most nested path.

@vkubiv vkubiv force-pushed the fix-bug-with-vp-referncing-presex branch 4 times, most recently from 02e9998 to ce5e1b3 Compare October 25, 2022 12:19
@codecov
Copy link

codecov bot commented Oct 25, 2022

Codecov Report

Merging #3417 (c2cd7f6) into main (368f53b) will increase coverage by 0.00%.
The diff coverage is 86.66%.

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

@@           Coverage Diff           @@
##             main    #3417   +/-   ##
=======================================
  Coverage   87.57%   87.58%           
=======================================
  Files         343      343           
  Lines       46809    46832   +23     
=======================================
+ Hits        40994    41017   +23     
  Misses       4318     4318           
  Partials     1497     1497           
Impacted Files Coverage Δ
pkg/doc/verifiable/credential.go 90.51% <ø> (ø)
pkg/vdr/httpbinding/resolver.go 72.72% <70.00%> (-0.90%) ⬇️
pkg/doc/presexch/api.go 81.98% <90.32%> (+3.32%) ⬆️
pkg/vdr/httpbinding/vdr.go 87.23% <100.00%> (+1.18%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vkubiv vkubiv force-pushed the fix-bug-with-vp-referncing-presex branch 5 times, most recently from c8f5499 to c2cd7f6 Compare October 25, 2022 13:41
@vkubiv vkubiv force-pushed the fix-bug-with-vp-referncing-presex branch from c2cd7f6 to 5c49eff Compare October 25, 2022 14:27
@vkubiv vkubiv force-pushed the fix-bug-with-vp-referncing-presex branch from 5c49eff to 0197294 Compare October 25, 2022 14:46
@fqutishat fqutishat merged commit bee1ddf into hyperledger-archives:main Oct 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants