-
Notifications
You must be signed in to change notification settings - Fork 11.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
_requireMinted() is in master but is not in v4.6 #3494
Comments
It was not part of 4.6 or any other version before that. The |
@Amxx one of my ERC721 contracts does not have the |
That is not a security concern. Don't worry |
For contracts/token/ERC721/ERC721.sol, it seems to be that _requireMinted() function was introduced in v.4.7.
https://github.com/OpenZeppelin/openzeppelin-contracts/blob/release-v4.7/contracts/token/ERC721/ERC721.sol
has requireMinted()
but https://github.com/OpenZeppelin/openzeppelin-contracts/blob/release-v4.6/contracts/token/ERC721/ERC721.sol
does not have requireMinted().
in master branch:
https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC721/ERC721.sol
The second line says that it is last updated v.4.6.
// OpenZeppelin Contracts (last updated v4.6.0) (token/ERC721/ERC721.sol)
But it doesn't have requireMinted().
Is this expected?
I assumed that it was a minor bug in master branch. Should be
// OpenZeppelin Contracts (last updated v4.7.0) (token/ERC721/ERC721.sol)
The text was updated successfully, but these errors were encountered: