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

feat: VC wallet contents - add/get/remove #2652

Merged
merged 1 commit into from
Mar 18, 2021

Conversation

sudeshrshetty
Copy link
Contributor

Signed-off-by: sudesh.shetty sudesh.shetty@securekey.com

@codecov
Copy link

codecov bot commented Mar 18, 2021

Codecov Report

Merging #2652 (2e53539) into main (dd56601) will increase coverage by 0.02%.
The diff coverage is 96.72%.

❗ Current head 2e53539 differs from pull request most recent head a4a4d90. Consider uploading reports for the commit a4a4d90 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2652      +/-   ##
==========================================
+ Coverage   88.34%   88.37%   +0.02%     
==========================================
  Files         264      265       +1     
  Lines       20654    20699      +45     
==========================================
+ Hits        18247    18292      +45     
  Misses       1415     1415              
  Partials      992      992              
Impacted Files Coverage Δ
pkg/client/vcwallet/profile.go 87.75% <87.50%> (+2.04%) ⬆️
pkg/client/vcwallet/client.go 100.00% <100.00%> (ø)
pkg/client/vcwallet/contents.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd56601...a4a4d90. Read the comment docs.

- fixed: UUID for dbnamespacing instead of user ID.
- added add/get/remove features for credential, collection, diddocument,
metadata, connection data models.
- closes hyperledger-archives#2651

Signed-off-by: sudesh.shetty <sudesh.shetty@securekey.com>
@sudeshrshetty sudeshrshetty requested a review from DRK3 March 18, 2021 19:41
@fqutishat fqutishat merged commit d7768a4 into hyperledger-archives:main Mar 18, 2021
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.

VC Wallet Content store - Add/Remove/Get
4 participants