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

Alfajores release 3 #200

Merged
merged 7 commits into from
Apr 22, 2024
Merged

Alfajores release 3 #200

merged 7 commits into from
Apr 22, 2024

Conversation

soloseng
Copy link
Contributor

@soloseng soloseng commented Apr 15, 2024

Description

Deployed and upgraded Alfajores contracts post audit && pre Mainnet release.

Other changes

  • Updated DefaultStrategy contract deploy script to reuse existing deployed AddressSortedLinkedList library
  • Added tooling to generate the following multisig proposal payloads needed to upgrade to v3
    • Payload to upgrade each contracts
    • Payload to set the pauser for each contract
    • Payload to set the MinCountOfActiveGroups to 3
    • Payload to add a new multisig owner

@soloseng soloseng self-assigned this Apr 15, 2024
@soloseng soloseng requested a review from a team April 15, 2024 22:01
await generateSetPauser(hre, "RebasedStakedCelo", destinations, values, payloads);

await generateSetMinCountOfActiveGroups(hre, destinations, values, payloads);
await generateMultiSigAddOwner(hre, args.ownerAddress!, destinations, values, payloads);
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we hardcode the owner so we cannot make mistake when running the command manually?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will hardcode

@soloseng soloseng merged commit 67fc64c into releases/3 Apr 22, 2024
3 checks passed
soloseng added a commit that referenced this pull request May 13, 2024
* updated the proposal expiration logic (#196)

* performed check on from group balance (#197)

* removed unused error (#198)

* Soloseng/audit-fix-clst-1 (#199)

* performed check on from group balance

* switch getCeloForGroup to public function

* Alfajores release 3 (#200)

* alfajores contract release

* updated deployed contract abis

* Updated deploy script to use existing libraby

* - updated DefaultStrategy main ABI
- Removed unused solc input

* tooling to generate MultiSig proposal payload for v3 contract upgrade

* ++ multisig edge case test

* hardcoded multisig signer address

* Soloseng/mainnet-release-3 (#201)

* updated to fix local testing that deploy a new contract instance

* updated test as `scheduleTransfer()` no longer allows transfers when `celoAvailableForGroup` is too low

* only set multisig impersonator when needed
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