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

wormchain: add slashing params vaa #4194

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

joelsmith-2019
Copy link
Collaborator

This PR adds a new Wormhole core governance action payload to Wormchain, allowing guardians to update the slashing parameters. I'm unsure where I should include documentation for how the payload is structured. If theres a specific README, please let me know and I'll add some documentation on it.

// Core Wormhole Module
Module [32]byte = "Core"

// Action index (6 for Slashing Params Update)
Action uint16 = 6

Chain uint16

// Signed Blocks Window (big-endian uint64)
SignedBlocksWindow uint64

// Min Signed Per Window (big-endian uint64)
MinSignedPerWindow uint64

// Downtime Jail Duration (big-endian uint64)
DowntimeJailDuration uint64

// Slash Fraction DoubleSign (big-endian uint64)
SlashFractionDoubleSign uint64

// Slash Fraction Downtime (big-endian uint64)
SlashFractionDowntime uint64

Ref to Cosmos SDK Slashing Params: Params

@joelsmith-2019 joelsmith-2019 self-assigned this Dec 16, 2024
Copy link
Contributor

@bruce-riley bruce-riley left a comment

Choose a reason for hiding this comment

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

I only reviewed the sdk stuff.

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