Closed brianmcmichael closed 1 year ago
Deployed Contract exception: None of the Tinlake Managers have a license specified on Mainnet in Etherscan. They do in Goerli. I don't think this is a blocker for the spell, but @amusingaxl might be something to alert the Centrifuge team about.
Apparently the license is not shown on Etherscan UI, but there is one declared in the code:
Deployed Contract exception: None of the Tinlake Managers have a license specified on Mainnet in Etherscan. They do in Goerli. I don't think this is a blocker for the spell, but @amusingaxl might be something to alert the Centrifuge team about.
Apparently the license is not shown on Etherscan UI, but there is one declared in the code:
Yeah the code looks the same so seems like something they did it differently in etherscan. Not sure if they can update the license to display correctly under other settings
, but might be worth trying. Either way, as the tinlake manager is not fully in scope, it is definitely not a blocker for us.
Description
MakerDAO December 9, 2022 Executive Spell
Contribution Checklist
(PE-<TICKET_NUMBER>)
Checklist
officeHours
modifier override30 days
unless otherwise specified)ETH_GAS="XXX" ETH_GAS_PRICE="YYY" make deploy
mainnet
contract on etherscanmake archive-spell
ormake date="YYYY-MM-DD" archive-spell
to make an archive directory and copyDssSpell.sol
,DssSpell.t.sol
,DssSpell.t.base.sol
, andDssSpellCollateralOnboarding.sol
squash and merge
this PR