Owner of 721 and 1155 fix

[robGG1997]

closes #701

There is no ownerOf function for ERC1155 because a token is not necessarily unique, it can have multiple owners. So I didn't implement it.
BatchOwnerOf already works with both types for tokenId so no need for a refactor there.

[RyRy79261]

@robGG1997

So with ERC1155, theres a concept of BalanceOf & BalanceOfBatch

This allows you to see how many of a token, by its ID, an account has. If you have set up the token by that ID to be a singular, unique, NFT. Then it would return a balance of 1.

If it is an NFT that you've issued a few of, say a badge for attending an event, multiple accounts could have a balance of 1.

If its gold in a game, then the balance would return more than 1 in some instances:
https://docs.openzeppelin.com/contracts/3.x/erc1155

[robGG1997]

@robGG1997

So with ERC1155, theres a concept of BalanceOf & BalanceOfBatch

This allows you to see how many of a token, by its ID, an account has. If you have set up the token by that ID to be a singular, unique, NFT. Then it would return a balance of 1.

If it is an NFT that you've issued a few of, say a badge for attending an event, multiple accounts could have a balance of 1.

If its gold in a game, then the balance would return more than 1 in some instances: https://docs.openzeppelin.com/contracts/3.x/erc1155

Understood, so I should refactor Balance function instead?

[RyRy79261]

@robGG1997 Yeah I rate just adding the non batch variant for 1155 if its missing is fine. I still need to actually review the code though.

[robGG1997]

@robGG1997 Yeah I rate just adding the non batch variant for 1155 if its missing is fine. I still need to actually review the code though.

Updated BalanceOf for Erc1155

[kantagara]

LGTM!

[juans-chainsafe]

Samples working! tested with wallet connect and web3auth.

Also fixed the issue with ImportNftTexture scene ✅