diff --git a/.changeset/old-countries-lie.md b/.changeset/old-countries-lie.md new file mode 100644 index 0000000000000..0713178a47991 --- /dev/null +++ b/.changeset/old-countries-lie.md @@ -0,0 +1,5 @@ +--- +'@eth-optimism/contracts-periphery': patch +--- + +Remove ownable upgradable from erc721 factory diff --git a/packages/contracts-periphery/contracts/universal/op-erc721/OptimismMintableERC721Factory.sol b/packages/contracts-periphery/contracts/universal/op-erc721/OptimismMintableERC721Factory.sol index 116e8a0d21649..914668ffd43eb 100644 --- a/packages/contracts-periphery/contracts/universal/op-erc721/OptimismMintableERC721Factory.sol +++ b/packages/contracts-periphery/contracts/universal/op-erc721/OptimismMintableERC721Factory.sol @@ -1,9 +1,6 @@ // SPDX-License-Identifier: MIT pragma solidity 0.8.15; -import { - OwnableUpgradeable -} from "@openzeppelin/contracts-upgradeable/access/OwnableUpgradeable.sol"; import { OptimismMintableERC721 } from "./OptimismMintableERC721.sol"; import { Semver } from "@eth-optimism/contracts-bedrock/contracts/universal/Semver.sol"; @@ -11,7 +8,7 @@ import { Semver } from "@eth-optimism/contracts-bedrock/contracts/universal/Semv * @title OptimismMintableERC721Factory * @notice Factory contract for creating OptimismMintableERC721 contracts. */ -contract OptimismMintableERC721Factory is Semver, OwnableUpgradeable { +contract OptimismMintableERC721Factory is Semver { /** * @notice Emitted whenever a new OptimismMintableERC721 contract is created. * @@ -23,12 +20,12 @@ contract OptimismMintableERC721Factory is Semver, OwnableUpgradeable { /** * @notice Address of the ERC721 bridge on this network. */ - address public bridge; + address public immutable bridge; /** * @notice Chain ID for the remote network. */ - uint256 public remoteChainId; + uint256 public immutable remoteChainId; /** * @notice Tracks addresses created by this factory. @@ -41,15 +38,6 @@ contract OptimismMintableERC721Factory is Semver, OwnableUpgradeable { * @param _bridge Address of the ERC721 bridge on this network. */ constructor(address _bridge, uint256 _remoteChainId) Semver(1, 0, 0) { - initialize(_bridge, _remoteChainId); - } - - /** - * @notice Initializes the factory. - * - * @param _bridge Address of the ERC721 bridge on this network. - */ - function initialize(address _bridge, uint256 _remoteChainId) public initializer { require( _bridge != address(0), "OptimismMintableERC721Factory: bridge cannot be address(0)" @@ -61,9 +49,6 @@ contract OptimismMintableERC721Factory is Semver, OwnableUpgradeable { bridge = _bridge; remoteChainId = _remoteChainId; - - // Initialize upgradable OZ contracts - __Ownable_init(); } /**