Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PoS: convert tokens to TM voting power when checking for changes #1083

Merged
merged 2 commits into from
Jan 25, 2023

Conversation

tzemanovic
Copy link
Member

backported from #740 to fix #1082

tzemanovic added a commit that referenced this pull request Jan 25, 2023
@tzemanovic tzemanovic marked this pull request as ready for review January 25, 2023 10:05
juped pushed a commit that referenced this pull request Jan 25, 2023
@juped juped force-pushed the tomas/fix-tm-voting-power-check branch from b22ec07 to 515dfcb Compare January 25, 2023 10:21
Copy link
Contributor

@cwgoes cwgoes left a comment

Choose a reason for hiding this comment

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

Minor nits. Code changes otherwise look fine but haven't tested.

proof_of_stake/src/lib.rs Outdated Show resolved Hide resolved
tzemanovic added a commit that referenced this pull request Jan 25, 2023
@tzemanovic tzemanovic force-pushed the tomas/fix-tm-voting-power-check branch from 515dfcb to 214c613 Compare January 25, 2023 10:35
@juped juped force-pushed the tomas/fix-tm-voting-power-check branch from 214c613 to b8a9221 Compare January 25, 2023 10:42
@juped juped self-requested a review January 25, 2023 11:02
@juped juped changed the base branch from main to maint-0.13 January 25, 2023 11:04
@juped juped merged commit d5745f7 into maint-0.13 Jan 25, 2023
juped added a commit that referenced this pull request Jan 25, 2023
* tag 'v0.13.3':
  Namada 0.13.3
  changelog: add #1083
  PoS: convert tokens to TM voting power when checking for changes
@juped juped deleted the tomas/fix-tm-voting-power-check branch January 25, 2023 11:30
bengtlofgren pushed a commit that referenced this pull request May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tendermint validator set updates don't respect tm_votes_per_token param
3 participants