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: rewards and rewards v2 integration for hello world #92

Merged
merged 31 commits into from
Jan 18, 2025

Conversation

Sidu28
Copy link
Contributor

@Sidu28 Sidu28 commented Oct 24, 2024

Routes createAVSRewardsSubmission call via the servicemanager called by the rewardsInitiator vs directly calling it.

@Sidu28 Sidu28 requested a review from stevennevins October 24, 2024 20:06
@stevennevins
Copy link
Collaborator

there is some more updates that I think we need to make this more flexible.
I started addressing some things in #93

we need to add a function with an initializer modifier on the service manager to initialize the owner and rewards updater state variables. this makes it so you don't need to prank(address(0));

these values need to be read from a config file for the hello world deployment

And then we need to improve the core deployment library to set the rewards updater address on the RewardsCoordinator. We can't read this from the config json files in the core contract repo. It looks like it's an address controlled by the person who wrote the script or maybe the pk for it is shared, but it's likely an internal address to eigenlayer and needs to be more configurable or a default anvil key

@Sidu28
Copy link
Contributor Author

Sidu28 commented Oct 28, 2024

there is some more updates that I think we need to make this more flexible. I started addressing some things in #93

we need to add a function with an initializer modifier on the service manager to initialize the owner and rewards updater state variables. this makes it so you don't need to prank(address(0));

these values need to be read from a config file for the hello world deployment

And then we need to improve the core deployment library to set the rewards updater address on the RewardsCoordinator. We can't read this from the config json files in the core contract repo. It looks like it's an address controlled by the person who wrote the script or maybe the pk for it is shared, but it's likely an internal address to eigenlayer and needs to be more configurable or a default anvil key

To make it more straightforward, should we just hardcode it in the CoreDeploymentLib.sol file to a default anvil key ourselves instead of using the config?

Sidu28 and others added 6 commits November 6, 2024 09:14
* fix: initialize sm and rewards coordinator config

* add rewards owner and initiator as inputs to deploy

* added helloWorld config

---------

Co-authored-by: kachapah <[email protected]>
Copy link
Collaborator

@stevennevins stevennevins left a comment

Choose a reason for hiding this comment

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

Looks good! Can you add a step in the README that runs through after the contracts are deployed is able to go through the payment flow. I realize we missed that on the first pass for this feature

@Sidu28 Sidu28 requested a review from stevennevins November 12, 2024 16:31
@Sidu28
Copy link
Contributor Author

Sidu28 commented Dec 26, 2024

@stevennevins wanted to bump this for review

Sidu28 and others added 7 commits January 6, 2025 05:30
* init

* builds

* updates

* updated commit

* added all functions

* added to readme

* rewards V2 working

* Update test.yml

* Update test.yml

---------

Co-authored-by: steven <[email protected]>
@stevennevins stevennevins changed the title Route createAVSPaymetnSubmission via SM feat: rewards integration for hello world Jan 17, 2025
@stevennevins stevennevins changed the title feat: rewards integration for hello world feat: rewards and rewards v2 integration for hello world Jan 17, 2025
@Sidu28 Sidu28 merged commit b8148c7 into master Jan 18, 2025
1 check passed
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