Skip to content

Coordinator Alpha 12#92

Merged
KPrasch merged 83 commits intomainfrom
development
Aug 28, 2023
Merged

Coordinator Alpha 12#92
KPrasch merged 83 commits intomainfrom
development

Conversation

@KPrasch
Copy link
Copy Markdown
Member

@KPrasch KPrasch commented Jun 26, 2023

No description provided.

cygnusv and others added 20 commits June 9, 2023 08:04
Also replace contract owner for parameter administration role
Initiator is just the entity that calls initiateRitual(), needs to be authorized, and provides initial payment.
Authority is who controls ritual lifecycle going forward.
Fee Models dictate what's the cost for rituals
Also, changes AccessControl base contract for the more specific AccessControlDefaultAdminRules
Also, change isInitiationRegulated to isInitiationPublic
Can't revoke INITIATOR_ROLE addresses since our AccessControl doesn't inherit from AccessControlEnumerable. It doesn't matter much anyway
We may want to parametrize tests based on this
Add also previous StakeInfo deployment on Mumbai
Either ritual is successful and fees can be used by the protocol, or ritual is failed and fees are refunded to initiator (consider partial refund)
@KPrasch KPrasch changed the title [WIP] Threshold access control application [WIP] Threshold access control application contracts Jun 26, 2023
@KPrasch KPrasch changed the title [WIP] Threshold access control application contracts [WIP] Coordinator Alpha 8 Jul 12, 2023
@KPrasch KPrasch changed the title [WIP] Coordinator Alpha 8 [WIP] Coordinator Alpha 9 Aug 22, 2023
@KPrasch KPrasch changed the title [WIP] Coordinator Alpha 9 [WIP] Coordinator Alpha 11 Aug 24, 2023
@KPrasch KPrasch changed the title [WIP] Coordinator Alpha 11 [WIP] Coordinator Alpha 12 Aug 26, 2023
Yet another batch of improvements for Coordinator
@KPrasch KPrasch changed the title [WIP] Coordinator Alpha 12 Coordinator Alpha 12 Aug 28, 2023
@KPrasch KPrasch marked this pull request as ready for review August 28, 2023 11:03
@KPrasch KPrasch merged commit 95de44f into main Aug 28, 2023
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.

5 participants