Currently, SPGNFT supports public minting as introduced in PR #16. However, this feature is not exposed in the periphery contracts. The periphery functions still enforce the onlyCallerWithMinterRole modifier, even though the SPGNFT contract supports public minting.
Suggested Solution
Modify (and rename) the onlyCallerWithMinterRole modifier to check for the isPublicMinting flag. If the flag is enabled, bypass the minter role check to allow public minting.
Definition of Done
Code is merged, tests pass, and functionality is verified.
Description and Context
Currently, SPGNFT supports public minting as introduced in PR #16. However, this feature is not exposed in the periphery contracts. The periphery functions still enforce the
onlyCallerWithMinterRolemodifier, even though the SPGNFT contract supports public minting.Suggested Solution
Modify (and rename) the
onlyCallerWithMinterRolemodifier to check for theisPublicMintingflag. If the flag is enabled, bypass the minter role check to allow public minting.Definition of Done