Skip to content

Commit

Permalink
Remove ERC20 from core contracts (#8212)
Browse files Browse the repository at this point in the history
### Description

`RegisteredContracts` list used in ContractKit for comprehensive registry lookup and governance proposal decoding (among others) errantly includes `CeloContract.ERC20` introduced in #7753 despite this not being a registered contract. This PR adds `CeloContract.ERC20` to `AuxiliaryContracts` to be filtered out.

### Tested

Before change, `Error: ERC20 not yet deployed for this chain` is emitted on many CLI changes. After change, error is no longer observed.
  • Loading branch information
yorhodes authored Jul 6, 2021
1 parent 7285db8 commit 569554f
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions packages/sdk/contractkit/src/base.ts
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ const AuxiliaryContracts = [
CeloContract.MultiSig,
CeloContract.MetaTransactionWalletDeployer,
CeloContract.MetaTransactionWallet,
CeloContract.ERC20,
]
export const RegisteredContracts = AllContracts.filter((v) => !AuxiliaryContracts.includes(v))

Expand Down

0 comments on commit 569554f

Please sign in to comment.