Skip to content

Commit

Permalink
fix: align token symbol with parameter naming
Browse files Browse the repository at this point in the history
This pr is to address issue base-org#99

Description
-----------
The `_l1erc721` parameter in createOptimismMintableERC721
represents the L1 token, while `L1T721` was incorrectly
used as the L2 token's symbol. Changed to `L2T721` to
properly reflect the relationship between the parameter
and the created token.
  • Loading branch information
0xObsidian committed Dec 3, 2024
1 parent ed36aac commit b463b16
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion script/deploy/l2/tests/DeployTestTokenContracts.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ contract DeployTestTokenContracts is Script {
console.log("Bridged erc20 deployed to: %s", address(erc20));

address erc721 = OptimismMintableERC721Factory(payable(Predeploys.OPTIMISM_MINTABLE_ERC721_FACTORY))
.createOptimismMintableERC721(_l1erc721, "L2 TEST ERC721", "L1T721");
.createOptimismMintableERC721(_l1erc721, "L2 TEST ERC721", "L2T721");
console.log("Bridged erc721 deployed to: %s", address(erc721));

vm.stopBroadcast();
Expand Down

0 comments on commit b463b16

Please sign in to comment.