web3ui / web3uikit

Lightweight reusable Web3 UI components for dapps.
https://web3uikit.com
MIT License
1.72k stars 269 forks source link

NFTBalance with ERC1155 and all <NFT/> options #724

Closed Kipperlenny closed 2 years ago

Kipperlenny commented 2 years ago

first: as far as I can see, I cannot use the options from <NFT> in the <NFTBalance> Tag right?

Second and more important: My NFTs are ERC1155, but they show up as ERC721. And I have more than one of them, but I cannot see any "count" on the cards...

AbhinavMV commented 2 years ago

Hey @Kipperlenny TY for using the kit

  1. Could you elaborate this question ?
  2. Could you share the the address, chainid etc ?? You can use NFT Balance component to see multiple NFTs on the address. Checkout our storybook - https://web3ui.github.io/web3uikit/?path=/story/1-web3-nft-balance--default
Kipperlenny commented 2 years ago

image

Rinkeby, https://rinkeby.etherscan.io/token/0x8267717011f667d46a85bc67667cb4c6f8ec8d4d

You can use NFT Balance component to see multiple NFTs on the address.

yes, but how can I use f.e. "fetchMetadata" from the <NFT> in the <NFTBalance> component?

https://web3ui.github.io/web3uikit/?path=/story/1-web3-nft--crypto-punk-01

AbhinavMV commented 2 years ago

yes, the contract type does seem to be a bug with the component. TY for letting us know 🙌🏻 We will be releasing our monorepo soon. And this will be fixed after that release 🙌🏻

fetchMetadata does not need to be used in the NFTBalance component since it uses Web3Api.account.getNFTs api from moralis to get the data.

AbhinavMV commented 2 years ago

Hey @Kipperlenny This issue is fixed in our latest monorepo. Give it a try. And let us know if you feel there is any issue

Closing this now.