feat: operator-sets-3-registration #279
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
ypatil12
reviewed
Jul 29, 2024
| * @param operator The address of the operator to deregister. | ||
| */ | ||
| function deregisterOperatorFromAVS(address operator) public virtual onlyRegistryCoordinator { | ||
| function deregisterOperatorFromAVS( |
There was a problem hiding this comment.
Should we still implement this function in our canonical example?
There was a problem hiding this comment.
gotcha. Were we envisioning that we would only have operator sets going forward, or support two ServiceManager contracts?
| * The off-chain service should do that validation separately | ||
| */ | ||
| function getRestakeableStrategies() external view returns (address[] memory) { | ||
| function getRestakeableStrategies() |
There was a problem hiding this comment.
We don't need this function anymore for operator set based AVSs, since this is provided via the MetadataURI
| view | ||
| returns (address[] memory) | ||
| { | ||
| function getOperatorRestakedStrategies( |
There was a problem hiding this comment.
Not needed for operator set release, since computed via operator<>operatorSet indexing along with metadataURI
stevennevins
changed the title
|
closed in favor of #301 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.