-
Notifications
You must be signed in to change notification settings - Fork 169
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
Update transactor for M2 mainnet contracts #194
Update transactor for M2 mainnet contracts #194
Conversation
return p | ||
} | ||
|
||
func NewG1Point(x, y *big.Int) *G1Point { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not used here, is it for the other PR? If so maybe move it over there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, I'm getting this is to be merged after #193?
tx, err := t.Bindings.BLSRegCoordWithIndices.RegisterOperatorWithCoordinator( | ||
|
||
// TODO(mooselumph): Fill out these | ||
kickParams := []regcoordinator.IRegistryCoordinatorOperatorKickParam{} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this to be done in following PR? can we create a ticket to track it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding a comment to the ticket for the churner updates: https://linear.app/eigenlabs/issue/EGDA-194/churner-updates-for-m2-mainnet-contracts#comment-66163e52
Why are these changes needed?
Checks