Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion .env.example
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,13 @@ FT_CONFIGURATOR=
FINAL_OWNER=
# Gnosis Safe Address and API key (Optional - Override default safe address for deployments, used with --safe when wiring)
PRIVATE_KEY_PROPOSER=
SAFE_API_KEY=

# Here we need to pick one or the other, right?
# If I use a delegate I don't need an owner, and if I use an owner I don't need a delegate.
Copy link
Collaborator

Choose a reason for hiding this comment

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

hmm iirc both are needed for the script on deployment otherwise they fail, but the delegate stuff was confusing for me too

SAFE_DELEGATE_ADDRESS=
SAFE_OWNER_ADDRESS=
SAFE_API_KEY=


# For verification of contracts on Etherscan, BscScan, SonicScan etc
ETHERSCAN_API_KEY=
1 change: 1 addition & 0 deletions DEPLOYMENT_CHECKLIST.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ Quick reference checklist for deploying FT token to production.
- [ ] Set `ETHERSCAN_API_KEY=<key>`
- [ ] Set `FT_CONFIGURATOR=<multisig-address>` role to have control of initial mint if network is sonic
- [ ] Set `FINAL_OWNER=<multisig-address>`
- [ ] Run `pnpm install`
- [ ] Run `pnpm compile`
- [ ] Run `pnpm test`

Expand Down
1 change: 1 addition & 0 deletions deploy/FT.ts
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ const deploy: DeployFunction = async (hre) => {
console.log(`Chain Config: ${chainConfig.name}`);
console.log(`Configurator: ${ftConfigurator} owner of the initial mint if network is sonic`);
console.log(`Endpoint V2: ${endpointV2Address}`);
// This confused me with safe delegate for the wiring. I guess there is a separate log in the wiring process.
Copy link
Collaborator

Choose a reason for hiding this comment

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

yes, delegate is term for LZ stuff too, not sure comment is needed for merge ?

seems like self notes : )

console.log(`Delegate: ${delegate}`);
console.log(`Final Owner: ${finalOwner}`);

Expand Down
10 changes: 5 additions & 5 deletions deployments/sonic/FT.json

Large diffs are not rendered by default.