Feat/nft collection

[adjisb]

New NFTCollection based on the old AvatarCollection:

• Added

  • OZ ERC2981.sol
  • batchMint(waveIndex, walletsAndAmounts) whenNotPaused onlyOwner so the admin can mint using less gas
  • waveMint(wallet, amount, waveIndex, signatureId, signature) whenNotPaused and cancelWave(waveIndex) onlyOwner so we can have multiple waves running at the same time.
  • safeBatchTransferFrom(from, to, ids, data) onlyAllowedOperator(from)
  • batchTransferFrom(from, to, ids) onlyAllowedOperator(from)
  • Custom errors
  • OZ v5 storage management
  • Fixed a known bug in ERC2771HandlerUpgradeable
  • Fixed the event emitted in setBaseURL
  • Changed the signature that was used in reveal so it is not the same as mint
  • Operator address in emitted events
  • We limit maxSupply by the amount of tokens minted (we ignore the burned tokens)

• Removed

  • The token Ids were generated with a pseudo-random, now they are sequential.
  • Access control we only have Ownable2StepUpgradeable
  • ERC721EnumerableUpgradeable we don’t need the enumerable part we use ERC721Upgradeable
  • unchecked/safe math because we use a newer version of solc
  • DefaultMintingValuesSet and all the related functions: setMarketingMint, setAllowlistMint and setPublicMint

• Kept

  • mint and personalize to be backward compatible with AvatarCollection
  • Burn tracking
  • OpenSea operator filterer now is disabled by default.
  • Pausable
  • We don’t have a way to revoke a signature

[openzeppelin-code]

Feat/nft collection

Generated at commit: 17c49f2de13cf2f43fb7c10edaee4e6ddef557b4

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	3 1 0 6 28 38

---

For more details view the full report in OpenZeppelin Code Inspector

[wojciech-turek]

A few questions and comments.