Skip to content
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

Implement TokensByOwner on base NFT contract #81

Closed
ethanfrey opened this issue Sep 10, 2020 · 2 comments · Fixed by #108
Closed

Implement TokensByOwner on base NFT contract #81

ethanfrey opened this issue Sep 10, 2020 · 2 comments · Fixed by #108
Milestone

Comments

@ethanfrey
Copy link
Member

ethanfrey commented Sep 10, 2020

Builds on #44

Add secondary indexes somehow and allow query on owner not just token id

Requires secondary indexes to do cleanly: CosmWasm/cosmwasm#468

@ethanfrey ethanfrey added this to the NFT milestone Sep 10, 2020
@ethanfrey
Copy link
Member Author

Support is slated for cosmwasm 0.11 with CosmWasm/cosmwasm#557

@ethanfrey
Copy link
Member Author

Actually done here with #108

This was referenced Oct 13, 2020
mikedotexe added a commit to mikedotexe/cw-nfts that referenced this issue Feb 20, 2022
Note this issue is fixed, hence adding the "x": CosmWasm/cw-plus#81
humanalgorithm pushed a commit to public-awesome/cw721-ibc that referenced this issue May 10, 2022
Note this issue is fixed, hence adding the "x": CosmWasm/cw-plus#81
seinj0312 added a commit to seinj0312/cw-nft that referenced this issue Jun 30, 2024
Note this issue is fixed, hence adding the "x": CosmWasm/cw-plus#81
koala1029 pushed a commit to koala1029/cw-nfts that referenced this issue Jul 18, 2024
Note this issue is fixed, hence adding the "x": CosmWasm/cw-plus#81
M-Killer-dev pushed a commit to M-Killer-dev/cosmwasm-nft that referenced this issue Sep 9, 2024
Note this issue is fixed, hence adding the "x": CosmWasm/cw-plus#81
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 a pull request may close this issue.

1 participant