Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Instead of signer address use signer pubkey #216

Merged
merged 4 commits into from
Apr 2, 2020
Merged

Conversation

0xAshish
Copy link
Contributor

  • Heimdall requires signer pubkey for validator join tx so adding it into the staked event.
  • validator struct still stores key extracted from pubkey
  • signer change also emits an event with pubkey so that Heimdall can listen and add that key.

@mankenavenkatesh
Copy link

Pls change stake.js accordingly for stake and signerchange calls.

Copy link

@mankenavenkatesh mankenavenkatesh left a comment

Choose a reason for hiding this comment

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

LGTM

@jdkanani jdkanani merged commit b8a9b4b into develop Apr 2, 2020
@jdkanani jdkanani deleted the signer-pubkey branch April 2, 2020 08:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants