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.

Sign up for GitHub

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

Jump to bottom

Impl registry coordinator #76

Merged
merged 118 commits into from
Jun 27, 2023
Merged

Impl registry coordinator #76

merged 118 commits into from
Jun 27, 2023

Conversation

gpsanant
Copy link
Contributor

implemented an initial version of the registry coordinator

gpsanant and others added 30 commits May 25, 2023 14:56
@gpsanant
add initial stake registry
8d6737d
@gpsanant
Merge branch 'multiquorums' into add-stake-registry
0798ffa
@gpsanant
everything but quorumNumbers
98c5e77
@Sidu28
added indexregistry
5161eab
@Sidu28
fixed error
518ef4e
@Sidu28
fixed error
eac96b1
@Sidu28
fixed error
bfae892
@Sidu28
fixed error
dedc642
@Sidu28
fixed error
2f8df64
@Sidu28
Merge branch 'multiquorums' of https://github.com/Layr-Labs/eigenlaye…
e0e17a0 
…r-contracts into indexRegistry
@Sidu28
modified interface
4d80a7e
@Sidu28
modified interface
9ae1af4
@Sidu28
fixed incorrect checl
3abc044
@Sidu28
added additional lower bound check to getters
de579ec
@Sidu28
added additional lower bound check to getters
6b760fb
@Sidu28
fixed _updateTotalOperatorHistory
235b8ec
@Sidu28
fixed index update in _updateOperatorIdToIndexHistory
ae519e7
@Sidu28
added blockNumber update for deregistering operator
eee0936
@Sidu28
added blockNumber update for deregistering operator
58ead13
@Sidu28
added comments
bf3a740
@Sidu28
added comments
5286912
@gpsanant
fix conflicts
47c7506
@gpsanant
Merge branch 'master' into add-stake-registry
7e15efb
@gpsanant
more work on stakeregistry
69f04fb
@gpsanant
further reuse of internal functions
cef971f
@gpsanant
fix total stake update logic
500b41f
@gpsanant
fix voteweigher interface
1fb084c
@Sidu28
removed random inut
ba56be4
@Sidu28
fixed error
b0ac3b1
@Sidu28
fixed error
8f8c912
@gpsanant
Copy link
Contributor Author

Addressed almost everything. Everything that wasn't a straightforward change is open for review

@ChaoticWalrus
Copy link
Contributor

This LGTM now, thanks for all the back-and-forth.

@gpsanant gpsanant marked this pull request as ready for review June 27, 2023 18:13
@gpsanant
Copy link
Contributor Author

This LGTM now, thanks for all the back-and-forth.

thank you too

@gpsanant gpsanant merged commit 609c55d into multiquorums Jun 27, 2023
8 checks passed
@gpsanant gpsanant deleted the impl-registry-coordinator branch June 27, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChaoticWalrus ChaoticWalrus ChaoticWalrus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gpsanant @ChaoticWalrus @Sidu28