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

Problem:(fix #1148) default deposit client-cli flow is complex #1158

Merged
merged 1 commit into from
Mar 3, 2020

Conversation

linfeng-crypto
Copy link
Contributor

solution:

  • remove deposit-amount command
  • use deposit amount as default
  • deposit inputs to a staking address in --advanced mode

@codecov
Copy link

codecov bot commented Mar 2, 2020

Codecov Report

Merging #1158 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1158      +/-   ##
==========================================
- Coverage   63.83%   63.78%   -0.05%     
==========================================
  Files         149      149              
  Lines       19886    19900      +14     
==========================================
- Hits        12694    12693       -1     
- Misses       7192     7207      +15
Impacted Files Coverage Δ
client-cli/src/command/transaction_command.rs 0% <0%> (ø) ⬆️
chain-core/src/tx/fee/mod.rs 87% <0%> (-0.5%) ⬇️

Copy link
Collaborator

@leejw51crypto leejw51crypto left a comment

Choose a reason for hiding this comment

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

lgtm

@tomtau
Copy link
Contributor

tomtau commented Mar 3, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 3, 2020

@bors bors bot merged commit 6b53f0d into crypto-com:master Mar 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants