Skip to content

fix undelegate losing tokens issue#4553

Merged
alexanderbez merged 2 commits intocosmos:masterfrom
coinexchain:master
Jun 15, 2019
Merged

fix undelegate losing tokens issue#4553
alexanderbez merged 2 commits intocosmos:masterfrom
coinexchain:master

Conversation

@helldealer
Copy link
Copy Markdown
Contributor

fix the issue
Undelegate may lose the delegator’s tokens when exceed the MaxEntries of unbonding delegations #4551

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 14, 2019

Codecov Report

Merging #4553 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4553      +/-   ##
==========================================
- Coverage   53.47%   53.46%   -0.02%     
==========================================
  Files         257      257              
  Lines       16168    16168              
==========================================
- Hits         8646     8644       -2     
- Misses       6876     6878       +2     
  Partials      646      646

Copy link
Copy Markdown
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

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

ACK

@alexanderbez alexanderbez added Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity. ready-for-review C:x/staking labels Jun 14, 2019
Copy link
Copy Markdown
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

thanks for the fix! mind adding a clog entry too?

@alexanderbez alexanderbez merged commit 941effc into cosmos:master Jun 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:x/staking Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants