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

Feat split wallet command #3276

Merged

Conversation

vkubiv
Copy link
Contributor

@vkubiv vkubiv commented Jul 14, 2022

Title:
Split wallet command.

Summary:

Factor walletlite.Command form vcwallet.Command. It can now be used in the lite version of an agent that does not require didcomm functionality.

@vkubiv vkubiv force-pushed the feat-split-wallet-command branch from 3b9b61c to 0795da1 Compare July 14, 2022 12:05
@codecov
Copy link

codecov bot commented Jul 14, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@2097e31). Click here to learn what that means.
The diff coverage is 87.90%.

@@           Coverage Diff           @@
##             main    #3276   +/-   ##
=======================================
  Coverage        ?   88.18%           
=======================================
  Files           ?      317           
  Lines           ?    42946           
  Branches        ?        0           
=======================================
  Hits            ?    37873           
  Misses          ?     3727           
  Partials        ?     1346           
Impacted Files Coverage Δ
pkg/controller/command/vcwallet/command.go 95.36% <ø> (ø)
pkg/controller/command/didcommwallet/command.go 87.85% <87.85%> (ø)
pkg/controller/rest/vcwallet/operation.go 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

@vkubiv vkubiv force-pushed the feat-split-wallet-command branch from 0795da1 to f680d63 Compare July 14, 2022 13:12
Copy link
Contributor

@sudeshrshetty sudeshrshetty left a comment

Choose a reason for hiding this comment

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

Did you forgot make an entry here or was it inten

pkg/controller/command/vcwallet/command.go Outdated Show resolved Hide resolved
pkg/controller/rest/vcwallet/operation.go Show resolved Hide resolved
pkg/controller/command/vcwallet/command.go Show resolved Hide resolved
@vkubiv vkubiv force-pushed the feat-split-wallet-command branch from 891e0e3 to a24b620 Compare July 25, 2022 08:45
Copy link
Contributor

@sudeshrshetty sudeshrshetty left a comment

Choose a reason for hiding this comment

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

@vkubiv please make changes as discussed.
vcwallet, didcommwallet

@vkubiv vkubiv force-pushed the feat-split-wallet-command branch from a24b620 to ec13ace Compare July 26, 2022 12:44
pkg/controller/command/didcommwallet/command.go Outdated Show resolved Hide resolved
pkg/controller/command/didcommwallet/command.go Outdated Show resolved Hide resolved
pkg/controller/command/didcommwallet/command.go Outdated Show resolved Hide resolved
pkg/controller/command/didcommwallet/models.go Outdated Show resolved Hide resolved
Signed-off-by: Volodymyr Kubiv <volodymyr.kubiv@euristiq.com>
@vkubiv vkubiv force-pushed the feat-split-wallet-command branch from f094059 to 55bfc82 Compare July 27, 2022 13:11
@sudeshrshetty sudeshrshetty merged commit c7f6766 into hyperledger-archives:main Jul 27, 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.

3 participants