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

NewCommittee proposal needs more info #3543

Closed
lehins opened this issue Jul 18, 2023 · 0 comments · Fixed by #3557
Closed

NewCommittee proposal needs more info #3543

lehins opened this issue Jul 18, 2023 · 0 comments · Fixed by #3557
Assignees
Labels
conway SanchoNet Minimal working version of Governance
Projects

Comments

@lehins
Copy link
Contributor

lehins commented Jul 18, 2023

We need to add a set for the previous committee and the epoch number when the new committee member should expire. Also the key role needs fixing. So, instead of current

https://github.com/input-output-hk/cardano-ledger/blob/319f0f3e5bb9ee8411f059f72e31dcd30a92afc4/eras/conway/impl/src/Cardano/Ledger/Conway/Governance/Procedures.hs#L304

  | NewCommittee
    !(Set (Credential 'CommitteeCold (EraCrypto era))) -- Old Committee
    !(Map (Credential 'CommitteeCold (EraCrypto era)) EpochNo) -- New Committee with expiration
    !Rational -- Quorum
@lehins lehins added this to the Unblock PlutusV3 milestone Jul 18, 2023
@lehins lehins added the conway label Jul 18, 2023
@lehins lehins added this to To do in Conway via automation Jul 18, 2023
@lehins lehins added the SanchoNet Minimal working version of Governance label Jul 18, 2023
@teodanciu teodanciu self-assigned this Jul 19, 2023
@teodanciu teodanciu moved this from To do to In progress in Conway Jul 20, 2023
@lehins lehins moved this from In progress to In review in Conway Jul 24, 2023
Conway automation moved this from In review to Done Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conway SanchoNet Minimal working version of Governance
Projects
Status: Done
Conway
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants