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

Multisig status logic follow-up #784

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Multisig status logic follow-up #784

merged 1 commit into from
Aug 23, 2022

Conversation

maurolacy
Copy link
Contributor

@maurolacy maurolacy commented Aug 19, 2022

#783 follow-up.

Changes the voting logic so that not only open proposals, but also approved and rejected proposals, can still be voted on before they expire.

This has no impact on the final proposal's status, but it allows voters to better reflect the tallies and results of a proposal's voting, for accounting, statistics and historical information.

@maurolacy maurolacy self-assigned this Aug 19, 2022
@maurolacy maurolacy force-pushed the 677-status-logic-2 branch 2 times, most recently from 34c58a9 to ee36fab Compare August 19, 2022 14:35
@maurolacy maurolacy changed the base branch from main to 677-status-logic August 19, 2022 14:35
Copy link
Contributor

@hashedone hashedone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Base automatically changed from 677-status-logic to main August 23, 2022 12:55
@maurolacy maurolacy merged commit 1f06cba into main Aug 23, 2022
@maurolacy maurolacy deleted the 677-status-logic-2 branch August 23, 2022 12:55
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.

2 participants