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

Prevent updating if proposal is too close to voting period start #202

Conversation

pscott
Copy link
Collaborator

@pscott pscott commented Jun 13, 2023

Modify updateProposal to make it revert if the block timestamp is too close to the beginning of the vote.

too close is defined as 15 minutes (can be chagned before we merge). It is currently a constant but we could decide to make it a setting and add some setter for it.

Closes #186

@pscott pscott requested a review from Orland0x June 13, 2023 15:12
@codecov
Copy link

codecov bot commented Jun 13, 2023

Codecov Report

Merging #202 (9404d2a) into main (9ce547c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #202   +/-   ##
=======================================
  Coverage   83.37%   83.37%           
=======================================
  Files          29       29           
  Lines         451      451           
  Branches      113      113           
=======================================
  Hits          376      376           
  Misses         57       57           
  Partials       18       18           
Impacted Files Coverage Δ
src/Space.sol 95.38% <100.00%> (ø)

@pscott
Copy link
Collaborator Author

pscott commented Jun 14, 2023

#186 (comment)

@pscott pscott closed this Jun 14, 2023
@Orland0x Orland0x deleted the audit_prevent_updating_if_proposal_is_to_close_to_voting_period branch October 11, 2023 10:37
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.

audit: Proposal Can Be Updated Just Before Voting Starts
1 participant