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

Refine commission rate change tx #1973

Merged
merged 3 commits into from
Oct 24, 2023
Merged

Conversation

brentstone
Copy link
Collaborator

@brentstone brentstone commented Oct 6, 2023

Describe your changes

Add some missing checks for validator commissione rate change tx and clean up some old code.

Indicate on which release or other PRs this topic is based on

v0.23.0

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

@Fraccaman Fraccaman mentioned this pull request Oct 16, 2023
tzemanovic added a commit that referenced this pull request Oct 18, 2023
* origin/brent/refine-commission-tx:
  check if rate > 1 in lib code
  changelog: add #1973
  add checks and simplify code
tzemanovic added a commit that referenced this pull request Oct 24, 2023
* origin/brent/refine-commission-tx:
  check if rate > 1 in lib code
  changelog: add #1973
  add checks and simplify code
@tzemanovic tzemanovic mentioned this pull request Oct 24, 2023
@brentstone brentstone merged commit 66a7567 into main Oct 24, 2023
13 of 14 checks passed
@brentstone brentstone deleted the brent/refine-commission-tx branch October 24, 2023 11:26
brentstone pushed a commit that referenced this pull request Nov 11, 2023
* origin/brent/refine-commission-tx:
  check if rate > 1 in lib code
  changelog: add #1973
  add checks and simplify code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants