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

feat: frax msig #3

Closed
wants to merge 3 commits into from
Closed

feat: frax msig #3

wants to merge 3 commits into from

Conversation

pegahcarter
Copy link
Collaborator

@pegahcarter pegahcarter commented Aug 15, 2024

Notable changes:

  • isConfirmed() has a named return variable.
  • For loops gas-golfed.
  • removeOwner() pops array (old contract subtracted length).
  • Modifier revert messages.
  • Simplified fallback.
  • constructor name (see original version for what I mean).

Original version: link

@pegahcarter
Copy link
Collaborator Author

Ignore changes to FraxFerry- they are residual formatting.

@pegahcarter
Copy link
Collaborator Author

Dropping as we are instead using the msig solution as provided by Tron.

@pegahcarter pegahcarter deleted the feat/msig branch September 18, 2024 09:06
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.

1 participant