Skip to content

Create ERC721REnumerable.sol#3

Open
alpereira7 wants to merge 3 commits into
erc721r:mainfrom
alpereira7:main
Open

Create ERC721REnumerable.sol#3
alpereira7 wants to merge 3 commits into
erc721r:mainfrom
alpereira7:main

Conversation

@alpereira7

Copy link
Copy Markdown

Add modified ERC721Enumerable extension, compatible with ERC721R. This can be useful for developers that want to keep track of ownership of tokens, wallet contents...

Add modified ERC721Enumerable extension, compatible with ERC721R
@RogerPodacter

Copy link
Copy Markdown
Contributor

Thanks! So this essentially copies the OZ implementation and changes the names? This seems fine. Note that ERC721r implements totalSupply() but no harm in reimplementing it here and probably less confusing

@alpereira7

alpereira7 commented Oct 18, 2022

Copy link
Copy Markdown
Author

Thanks! So this essentially copies the OZ implementation and changes the names? This seems fine. Note that ERC721r implements totalSupply() but no harm in reimplementing it here and probably less confusing

I needed to get the NFTs owned by a wallet, which I couldn't find in ERC721r.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants