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

feat: update middleware submodule to v1.0.3-testnet-slashing release #478

Open
wants to merge 44 commits into
base: dev
Choose a base branch
from

Conversation

MegaRedHand
Copy link
Contributor

@MegaRedHand MegaRedHand commented Jan 27, 2025

What Changed?

This PR updates the middleware submodule to the v1.0.3-testnet-slashing release. As part of this:

  • adds the new RegistrationType prefix to the operator registration parameters (with an encoding test)
  • adds the SlashingRegistryCoordinator and SocketRegistry to the bindings generated
  • fixes a path typo on the bindings generation
  • makes the SDK compatible with the new SlashingRegistryCoordinator (tests pending) by making the service manager address optional
  • updates some type imports that were moved from contract interfaces to type-specific interfaces
  • bumps solc pinned version to 0.8.27 on all contracts
  • adds a LegacyRegistryCoordinator with additional functions to go back to M2 state

What's left for following PRs:

  • testing of new contracts: SlashingRegistryCoordinator, and LegacyRegistryCoordinator with enabled operator sets and disabled M2 quorums.

Reviewer Checklist

  • Code is well-documented
  • Code adheres to Go naming conventions
  • Code deprecates any old functionality before removing it

@MegaRedHand MegaRedHand marked this pull request as ready for review January 30, 2025 15:20
@MegaRedHand MegaRedHand changed the title feat: add type prefix to operator registration params feat: update middleware submodule to v1.0.3-testnet-slashing release Jan 30, 2025
@MegaRedHand MegaRedHand self-assigned this Jan 30, 2025
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