Skip to content

[mainnet] Deploy the newly updated Smart Escrow Contract#180

Merged
anikaraghu merged 5 commits into
mainfrom
smart-escrow-deploy
Jul 27, 2024
Merged

[mainnet] Deploy the newly updated Smart Escrow Contract#180
anikaraghu merged 5 commits into
mainfrom
smart-escrow-deploy

Conversation

@stevieraykatz

Copy link
Copy Markdown
Member

The escrow contract was updated with this PR: base/contracts#81
And audited in this PR: base/contracts#85

NESTED_SAFE=0x0a7361e734cf3f0394B0FC4a45C74E7a4Ec70940
START=1720674000
CLIFF_START=1724976000
END=1882764000

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Needs to be July 11, 2029 : 1878462000

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated to reflect updated END time.

@stevieraykatz stevieraykatz marked this pull request as ready for review June 24, 2024 19:40
include .env

.PHONY: deploy-new-implementation
deploy-new-implementation:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

worth just adding a --verify to see if that works and don't have to go through the separate verification step

Comment thread mainnet/2024-06-05-reredeploy-smart-escrow/script/DeploySmartEscrow.s.sol Outdated
anikaraghu
anikaraghu previously approved these changes Jun 24, 2024

@anikaraghu anikaraghu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM thanks! checked agains that reference you had put together for reviewers - btw can you also add the cliffStart to it?

BENEFICIARY_OWNER=0x6e1DFd5C1E22A4677663A81D24C6BA03561ef0f6
NESTED_SAFE=0x0a7361e734cf3f0394B0FC4a45C74E7a4Ec70940
START=1720674000
CLIFF_START=1724976000

@anikaraghu anikaraghu Jun 24, 2024

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is Fri Aug 30 2024 00:00:00 GMT+0000 - what should I compare that against?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Confirmed by Ben, based on date of first token received at Nested Multisig: https://optimistic.etherscan.io/tx/0x6d771624ecbff8704edfc5cbf3bff57b8bac9584775457b44e9872081cd0a733

…scrow.s.sol

Co-authored-by: anikaraghu <anika.raghuvanshi@gmail.com>
@cb-heimdall cb-heimdall dismissed anikaraghu’s stale review June 25, 2024 00:05

Approved review 2136966412 from anikaraghu is now dismissed due to new commit. Re-request for approval.

@cb-heimdall

cb-heimdall commented Jul 27, 2024

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
@anikaraghu anikaraghu merged commit 7f04a7e into main Jul 27, 2024
@anikaraghu anikaraghu deleted the smart-escrow-deploy branch July 27, 2024 00:51
@stevieraykatz

Copy link
Copy Markdown
Member Author

🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants