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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

wip: mainnet deploy configs #694

Merged
merged 6 commits into from
Sep 8, 2021
Merged

wip: mainnet deploy configs #694

merged 6 commits into from
Sep 8, 2021

Conversation

prestwich
Copy link
Contributor

No description provided.

@prestwich
Copy link
Contributor Author

need to replace the governor address with msig

@prestwich prestwich added the typescript Typescript dev work required label Sep 6, 2021
@prestwich prestwich marked this pull request as ready for review September 8, 2021 20:01

export const config: CoreConfig = {
environment: 'prod',
updater: '0xDB2091535eb0Ee447Ce170DDC25204FEA822dd81',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Verified, correct

environment: 'prod',
updater: '0xDB2091535eb0Ee447Ce170DDC25204FEA822dd81',
recoveryTimelock: 60 * 60 * 24, // 1 day
recoveryManager: '0x3D9330014952Bf0A3863FEB7a657bfFA5C9D40B9',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@barbaraliau verify

Copy link
Contributor

Choose a reason for hiding this comment

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

verified

recoveryTimelock: 60 * 60 * 24, // 1 day
recoveryManager: '0x3D9330014952Bf0A3863FEB7a657bfFA5C9D40B9',
optimisticSeconds: 60 * 60 * 3, // 3 hours
watchers: ['0xeE42B7757798cf495CDaA8eDb0CC237F07c60C81'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Verified, correct

watchers: ['0xeE42B7757798cf495CDaA8eDb0CC237F07c60C81'],
governor: {
domain: chainJson.domain,
address: '0x5Fa96B622D1F4e920b92040c10fA297ca496ad37',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@barbaraliau verify

Copy link
Contributor

Choose a reason for hiding this comment

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

verified

Copy link
Contributor

@yourbuddyconner yourbuddyconner left a comment

Choose a reason for hiding this comment

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

Watcher and Updater attest keys confirmed

Copy link
Contributor

@barbaraliau barbaraliau left a comment

Choose a reason for hiding this comment

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

Verified recoveryManager and governor addresses

watchers: ['0xeE42B7757798cf495CDaA8eDb0CC237F07c60C81'],
governor: {
domain: chainJson.domain,
address: '0x5Fa96B622D1F4e920b92040c10fA297ca496ad37',
Copy link
Contributor

Choose a reason for hiding this comment

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

verified

environment: 'prod',
updater: '0xDB2091535eb0Ee447Ce170DDC25204FEA822dd81',
recoveryTimelock: 60 * 60 * 24, // 1 day
recoveryManager: '0x3D9330014952Bf0A3863FEB7a657bfFA5C9D40B9',
Copy link
Contributor

Choose a reason for hiding this comment

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

verified

environment: 'prod',
updater: '0xDB2091535eb0Ee447Ce170DDC25204FEA822dd81',
recoveryTimelock: 60 * 60 * 24, // 1 day
recoveryManager: '0x3D9330014952Bf0A3863FEB7a657bfFA5C9D40B9',
Copy link
Contributor

Choose a reason for hiding this comment

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

verified

environment: 'prod',
updater: '0xDB2091535eb0Ee447Ce170DDC25204FEA822dd81',
recoveryTimelock: 60 * 60 * 24, // 1 day
recoveryManager: '0x3D9330014952Bf0A3863FEB7a657bfFA5C9D40B9',
Copy link
Contributor

Choose a reason for hiding this comment

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

verified

@prestwich prestwich merged commit 33a8592 into main Sep 8, 2021
@prestwich prestwich deleted the prestwich/mainnet-configs branch September 17, 2021 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typescript Typescript dev work required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants