solidstate-network/solidstate-solidity

ERC1155 _mint without data

Closed this issue · 2 comments

It is unclear what the ERC1155 _mint data payload is used for. In many cases it would save gas to include a new internal function to avoid passing this argument:

function _mint (
  address account,
  uint256 tokenId,
  uint256 amount
) internal {
  _mint(account, tokenId, amount, '');
}

@zimmah What is the data payload used for?

Include __internal test in ERC1155Base.ts.

In combination with the "safe/unsafe" and "single/batch" transfer functions, this might require more code than it's worth to be comprehensive and intuitive.