Skip to content

feat: OperatorStateRetrieverWithSocket #457

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

pakim249CAL
Copy link

@pakim249CAL pakim249CAL commented Apr 7, 2025

Motivation:

Add getOperatorStateWithSocket entry points to negate the need for an additional query for the socket for offchain integrators.

This change was merged to eigenda-dev. However, this should also probably be propagated over to here as well. That way, eigenda can update to a canonical branch in this repo.

Modifications:

OperatorStateRetriever

  • Added getOperatorStateWithSocket(
    ISlashingRegistryCoordinator registryCoordinator,
    bytes32 operatorId,
    uint32 blockNumber)
  • Added getOperatorStateWithSocket(
    ISlashingRegistryCoordinator registryCoordinator,
    bytes memory quorumNumbers,
    uint32 blockNumber)

Result:

The OperatorStateRetriever will have these additional functions.

@pakim249CAL pakim249CAL changed the title feat: operatorStateWithSocket (#418) OperatorStateRetrieverWithSocket Apr 7, 2025
**Motivation:**

*Explain here the context, and why you're making that change. What is
the problem you're trying to solve.*

**Modifications:**

*Describe the modifications you've done.*

**Result:**

*After your change, what will change.*

---------

Co-authored-by: Patrick Kim <[email protected]>

forge: fmt
@pakim249CAL pakim249CAL force-pushed the eigenda-dev-rebased branch from a3879ff to d477b66 Compare April 7, 2025 16:05
@pakim249CAL pakim249CAL changed the title OperatorStateRetrieverWithSocket feat: OperatorStateRetrieverWithSocket Apr 7, 2025
@pakim249CAL pakim249CAL requested a review from wadealexc April 7, 2025 16:21
@pakim249CAL pakim249CAL marked this pull request as ready for review April 7, 2025 16:21
@RonTuretzky
Copy link
Contributor

bumping as this will make commonware intergation easier

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.

3 participants