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

wip: update eigenlayer-middleware #47

Closed
wants to merge 1 commit into from

Conversation

haardikk21
Copy link

@haardikk21 haardikk21 commented Apr 26, 2024

Solves #43

Updates EL Middleware dependency with the following:

  1. Updates IncredibleSquaringDeployer.s.sol to read avsDirectory address and pass it to the ServiceManager as required
  2. Instantiates the ERC-20 strategy with values set for thirdPartyTransfersForbiddenValues
  3. Updates the deploy-eigenlayer-save-anvil-state.sh script to point to the updated paths for the Deployer

I set this PR as draft since I have been unable to fully test my changes due to an issue in Foundry already reported by @samlaf here - foundry-rs/foundry#7502

@samlaf
Copy link
Collaborator

samlaf commented Apr 27, 2024

@haardikk21 thanks for giving this a try!
I pretty much ended up doing same thing as you, except had to change to using dockerized anvil to be able to pin it at an older version to solve the problem you're facing.
Also needed to update eigensdk to get latest contract changes when registering operators.
Check #46

@haardikk21
Copy link
Author

haardikk21 commented Apr 27, 2024

@haardikk21 thanks for giving this a try!

I pretty much ended up doing same thing as you, except had to change to using dockerized anvil to be able to pin it at an older version to solve the problem you're facing.

Also needed to update eigensdk to get latest contract changes when registering operators.

Check #46

Thanks for #46 - it was fun, gives me a chance to explore deeper as we recently started work on our AVS

I'll close this PR in the morning (doesn't let me do it on GitHub mobile) - or you can just close it

@haardikk21 haardikk21 closed this Apr 27, 2024
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