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.

Sign up for GitHub

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

Jump to bottom

20240328 deployment gc #125

Merged
merged 22 commits into from
Apr 2, 2024
Merged

20240328 deployment gc #125

merged 22 commits into from
Apr 2, 2024

Conversation

benjaminbollen
Copy link
Member

  • turns out most important setting was: paying a negligible priority fee on mainnet for the deployment to reliably succeed.
  • before I got there, hardened the deployment script.
Gnosis Chain deployment
=================
Deployment Date: 2024-04-01 17:51:01
Version: 0.3.1-alpha
Git Commit: 1fd0982c4c86533de2845a975113bcd763df8a56
Deployer Address: 0x7619F26728Ced663E50E578EB6ff42430931564c, Initial nonce: 8
Compiler Version: v0.8.23+commit.f704f362

Deployed Contracts:
Hub: 0x00315615f899F90d6Bc46EaaC911E37871d62DD8
Migration: 0xE22bFb93aDFa2b446226Ffb6FDa8dC641C819f23
NameRegistry: 0xa28f05bd1C054E2AA9680497e03Df2cc17f1b3d8
ERC20Lift: 0xE2CD16612EaF37432a8384c5dA8a66e1CAf0E68F
StandardTreasury: 0x8e9b584764C467d95C23c27c0Ed0eE96A95b427d
BaseGroupMintPolicy: 0x6ecbCB0166A8143aa4AC19a43Ea31f915D3D36dB
MastercopyDemurrageERC20: 0x16920293865aCe1c5e97e672eaECFD3254224b6f
MastercopyInflationaryERC20: 0x3467E9977856A422CfE7960D0a34af2FFB91B47f
MastercopyStandardVault: 0xc70FAfea358d371477BbAd1c3adE0936AC4eEe24

benjaminbollen and others added 22 commits March 22, 2024 21:20
@benjaminbollen
(hub, circles): calculate issuance must also update v1 minting status
844b55b
@benjaminbollen
(circles): internal calculate issuance can remain view function
19eccf4
@benjaminbollen
(test/hub): start on v1 status checks test for calculate/mint issuance
2e9526c
@benjaminbollen
(test): git submodule contracts v1
05d2f95
@benjaminbollen
(test): rather build on Mock HubV1
5233faa
@jaensen
fixed typo: calculateIssuace -> calculateIssuance
54f586f
@benjaminbollen
Merge branch '20240320-deployment' into 20240322-bug-calculate-issuan…
eca45c9 
…ce-must-update-v1status
@benjaminbollen
(hub, circles): personalMint should not revert, calculateIssuanceWith…
88419a2 
…Check
@benjaminbollen
Merge remote-tracking branch 'origin/develop' into 20240322-bug-calcu…
95d0e67 
…late-issuance-must-update-v1status
@benjaminbollen
(test): restart on mocking hubs for migration
552182e
@benjaminbollen
(test): rebuild Circles v1 Mock HubV1 and MockToken in tests
120ad5c
@benjaminbollen
(test): fix interface of trust
ad04407
@benjaminbollen
(test): neither create2 or create work in the test setting; revert an…
b854ed3 
…d instead mock to set addresses later than constructor
@benjaminbollen
(test/migration): initial register of V1 Alice
809dc2f
@benjaminbollen
(tests): test consecutive mint periods and migration WIP
23ecf4d
@benjaminbollen
(test/Circles): test issuance, consecutive periods
53b2741
@benjaminbollen
(test/hub): improve hub transfer test too with relative equality
fb5f449
@benjaminbollen
(test/Circles): test flow to invite v1 user to v2
4de6630
@benjaminbollen
(test): review edit in comment
77c8d94
@benjaminbollen
(Name): for pre-release rename to to easily differentiate
a5548e3
@benjaminbollen
(deploy): update script for Gnosis Chain deployment
1fd0982
@benjaminbollen
(deploy): finally a deployment on Gnosis Chain, solution: pay priorit…
b996e30 
…y fees
@benjaminbollen
Copy link
Member Author

⚠️ first review and merge #123 @jaensen

@benjaminbollen benjaminbollen linked an issue Apr 1, 2024 that may be closed by this pull request
Deployment #98
Closed
3 tasks
jaensen
jaensen approved these changes Apr 2, 2024
View reviewed changes
Copy link
Contributor

@jaensen jaensen left a comment

Choose a reason for hiding this comment

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

lgtm

@jaensen jaensen merged commit fca18a8 into develop Apr 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaensen jaensen jaensen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
circles-contracts-v2
Status: 💪Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Deployment
2 participants
@benjaminbollen @jaensen