You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
🧐 Motivation
The existing ERC721Component initializer registers both IERC721_ID and IERC721_METADATA_ID, while the IERC721Metadata is described as optional in the EIP. Users could choose not to implement that interface, but if they call the given initializer, this will lead to an inconsistency that could break some UIs.
📝 Details
A second initializer can be added to be used in this case (not registering the IERC721_METADATA_ID). In-code comments and documentation must be detailed about the potential issue and when to use each initializer.
The text was updated successfully, but these errors were encountered:
🧐 Motivation
The existing ERC721Component initializer registers both IERC721_ID and IERC721_METADATA_ID, while the IERC721Metadata is described as optional in the EIP. Users could choose not to implement that interface, but if they call the given initializer, this will lead to an inconsistency that could break some UIs.
📝 Details
A second initializer can be added to be used in this case (not registering the IERC721_METADATA_ID). In-code comments and documentation must be detailed about the potential issue and when to use each initializer.
The text was updated successfully, but these errors were encountered: