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

feat: wallet.Issue supports jwtvc, verifiable.Credential can hold a jwtvc #3345

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

Moopli
Copy link
Contributor

@Moopli Moopli commented Aug 23, 2022

Signed-off-by: Filip Burlacu filip.burlacu@securekey.com

@codecov
Copy link

codecov bot commented Aug 23, 2022

Codecov Report

Merging #3345 (1339bf6) into main (07695f2) will decrease coverage by 0.00%.
The diff coverage is 90.32%.

@@            Coverage Diff             @@
##             main    #3345      +/-   ##
==========================================
- Coverage   87.89%   87.89%   -0.01%     
==========================================
  Files         330      330              
  Lines       45498    45571      +73     
==========================================
+ Hits        39992    40056      +64     
- Misses       4079     4085       +6     
- Partials     1427     1430       +3     
Impacted Files Coverage Δ
pkg/doc/verifiable/credential.go 90.11% <78.04%> (-0.85%) ⬇️
pkg/doc/verifiable/credential_jwt.go 86.36% <100.00%> (+0.31%) ⬆️
pkg/doc/verifiable/embedded_proof.go 83.01% <100.00%> (+0.32%) ⬆️
pkg/doc/verifiable/presentation.go 87.07% <100.00%> (+0.08%) ⬆️
pkg/wallet/wallet.go 98.23% <100.00%> (+0.14%) ⬆️

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

@Moopli Moopli force-pushed the jwt-wallet branch 4 times, most recently from 8418415 to 4c7da70 Compare August 24, 2022 06:00
@Moopli Moopli marked this pull request as ready for review August 24, 2022 06:05
@Moopli Moopli force-pushed the jwt-wallet branch 4 times, most recently from 0a2dd14 to 3491917 Compare August 24, 2022 22:28
…wtvc

Signed-off-by: Filip Burlacu <filip.burlacu@securekey.com>
@sudeshrshetty sudeshrshetty merged commit 6c0c224 into hyperledger-archives:main Aug 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.

None yet

2 participants