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

increased validator bond shares during delegation #24

Merged
merged 3 commits into from
Aug 10, 2023

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented Aug 1, 2023

  • Increased validator bond shares when delegating if the delegation was already a validator bond
  • Moved SharesFromTokens function calls to under if statements
  • Added unit test

@sampocs
Copy link
Collaborator Author

sampocs commented Aug 3, 2023

@xlab For some reason, I can't add you as a reviewer - but would be great to get your eyes on this!

@xlab
Copy link

xlab commented Aug 10, 2023

@sampocs lgtm, I've updated our tests and checked that it works in our E2E setup as well

@sampocs sampocs merged commit ac34f8b into v0.45.16-ics-lsm Aug 10, 2023
23 of 30 checks passed
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.

3 participants