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

feat: new common wallet package #2670

Merged
merged 1 commit into from Mar 24, 2021

Conversation

sudeshrshetty
Copy link
Contributor

@sudeshrshetty sudeshrshetty commented Mar 24, 2021

  • all common internal source inside pkg/client/vcwallet moved to
    pkg/wallet. This new package will be reused by upcoming command
    controllers.
  • vcwallet.New() to unlock wallet during instantiation, where existing
    instances of same wallet will be locked.
  • some refactoring for SDK binding for better client experience.
  • Part of Aries Framework Go wallet #2433

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

@sudeshrshetty sudeshrshetty changed the title feat: new wallet package feat: new common wallet package Mar 24, 2021
@codecov
Copy link

codecov bot commented Mar 24, 2021

Codecov Report

Merging #2670 (7d1c1c6) into main (074644c) will increase coverage by 0.03%.
The diff coverage is 97.71%.

❗ Current head 7d1c1c6 differs from pull request most recent head 1c9f654. Consider uploading reports for the commit 1c9f654 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2670      +/-   ##
==========================================
+ Coverage   88.36%   88.39%   +0.03%     
==========================================
  Files         265      266       +1     
  Lines       21130    21172      +42     
==========================================
+ Hits        18671    18715      +44     
+ Misses       1433     1431       -2     
  Partials     1026     1026              
Impacted Files Coverage Δ
pkg/wallet/contents.go 100.00% <ø> (ø)
pkg/wallet/profile.go 87.75% <ø> (ø)
pkg/wallet/wallet.go 97.24% <97.24%> (ø)
pkg/client/vcwallet/client.go 100.00% <100.00%> (+4.37%) ⬆️
pkg/wallet/kmsclient.go 88.17% <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 074644c...1c9f654. Read the comment docs.

- all common internal source inside `pkg/client/vcwallet` moved to
`pkg/wallet`. This new package will be reused by upcoming command
controllers.
- `vcwallet.New()` to unlock wallet during instantiation, where existing
instances of same wallet will be locked.
- some refactoring for SDK binding for better client experience.
- Part of hyperledger-archives#2433

Signed-off-by: sudesh.shetty <sudesh.shetty@securekey.com>
@troyronda troyronda merged commit 76a193d into hyperledger-archives:main Mar 24, 2021
llorllale pushed a commit to llorllale/agent-sdk that referenced this pull request Mar 25, 2021
Changes:

* refactored VDR.Create(): hyperledger-archives/aries-framework-go#2665
* support for non-deterministic keys in storage FormattedProvider:
hyperledger-archives/aries-framework-go#2667
* fixed didexchange.Client.CreateConnect():
hyperledger-archives/aries-framework-go#2670

Signed-off-by: George Aristy <george.aristy@securekey.com>
sudeshrshetty pushed a commit to sudeshrshetty/agent-sdk that referenced this pull request Aug 31, 2021
Changes:

* refactored VDR.Create(): hyperledger-archives/aries-framework-go#2665
* support for non-deterministic keys in storage FormattedProvider:
hyperledger-archives/aries-framework-go#2667
* fixed didexchange.Client.CreateConnect():
hyperledger-archives/aries-framework-go#2670

Signed-off-by: George Aristy <george.aristy@securekey.com>
sudeshrshetty added a commit to sudeshrshetty/aries-framework-go that referenced this pull request Oct 18, 2021
- all common internal source inside `pkg/client/vcwallet` moved to
`pkg/wallet`. This new package will be reused by upcoming command
controllers.
- `vcwallet.New()` to unlock wallet during instantiation, where existing
instances of same wallet will be locked.
- some refactoring for SDK binding for better client experience.
- Part of hyperledger-archives#2433

Signed-off-by: sudesh.shetty <sudesh.shetty@securekey.com>
sudeshrshetty added a commit to sudeshrshetty/aries-framework-go that referenced this pull request Jan 22, 2022
- all common internal source inside `pkg/client/vcwallet` moved to
`pkg/wallet`. This new package will be reused by upcoming command
controllers.
- `vcwallet.New()` to unlock wallet during instantiation, where existing
instances of same wallet will be locked.
- some refactoring for SDK binding for better client experience.
- Part of hyperledger-archives#2433

Signed-off-by: sudesh.shetty <sudesh.shetty@securekey.com>
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

3 participants