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

fix(drive): ValidatorSetUpdate doesn't contain nodeAddress #155

Merged
merged 1 commit into from
Dec 24, 2021

Conversation

shuplenkov
Copy link
Collaborator

@shuplenkov shuplenkov commented Dec 24, 2021

Issue being fixed or feature implemented

ValidatorSetUpdate doesn't contain nodeAddress

What was done?

Updated @dashevo/abci

How Has This Been Tested?

With tests

Breaking Changes

No

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shuplenkov shuplenkov added this to the v0.22.0 milestone Dec 24, 2021
@shumkov shumkov changed the title fix(drive): ValidatorSetUpdate doesn't contain nodeAddress fix(drive): ValidatorSetUpdate doesn't contain nodeAddress Dec 24, 2021
@shumkov shumkov changed the title fix(drive): ValidatorSetUpdate doesn't contain nodeAddress fix(drive): ValidatorSetUpdate doesn't contain nodeAddress Dec 24, 2021
Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@shuplenkov shuplenkov merged commit 951dd19 into v0.22-dev Dec 24, 2021
@shuplenkov shuplenkov deleted the update-abci branch December 24, 2021 13:22
@shuplenkov shuplenkov restored the update-abci branch December 24, 2021 13:22
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.

2 participants