Skip to content

Conversation

@sneakzttv
Copy link
Contributor

@sneakzttv sneakzttv commented Dec 6, 2024

Added an NFT tracker to the web3auth GUI for token streaming.

image

image

Testing:
Log in with web3 auth.
Open the wallet.
Press the toggle button on custom tokens to see the Nft menu.
Add a 1155 NFT.
See NFT balance populate.

https://gyazo.com/1ed9cfe1ac3aa68eee75ec41792ae574

Nft tracker
Add nft update
Toggle added
fixes
@sneakzttv sneakzttv added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Dec 6, 2024
@sneakzttv
Copy link
Contributor Author

sneakzttv commented Dec 6, 2024

Docs do not need to be completed as it is self explanatory, if we think we need it I can amend.

@sneakzttv sneakzttv added the documented Apply on Pull Request after documentation to allow merge label Dec 6, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script itself is becoming pretty unmanageable, can you make a separate script for this?

Copy link
Contributor Author

@sneakzttv sneakzttv Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, no problem. I should have condensed the functions to reduce UI elements but segregation works also.

Copy link
Contributor Author

@sneakzttv sneakzttv Dec 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

complete!

Copy link
Contributor

@juans-chainsafe juans-chainsafe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should work with any ERC-1155? Im trying this NFT https://testnets.opensea.io/assets/sepolia/0xAdBa07EE1940f1dEC04b1981C3602EbFef012486/1
But 0 is displayed
Screenshot 2024-12-06 at 2 56 07 PM

Screenshot 2024-12-06 at 2 55 34 PM

@sneakzttv
Copy link
Contributor Author

sneakzttv commented Dec 6, 2024

Should work with any ERC-1155? Im trying this NFT https://testnets.opensea.io/assets/sepolia/0xAdBa07EE1940f1dEC04b1981C3602EbFef012486/1 But 0 is displayed Screenshot 2024-12-06 at 2 56 07 PM

Screenshot 2024-12-06 at 2 55 34 PM

yes, make sure your rpc is up to date to the latest block, some of the sepolia ones are slow.

<(^.^<)

We should consider shifting test networks for efficiency. B3 is fast and easy to get testnet tokens, i vote there.

Segregated Custom token and Custom Nft managers for readability & performance.
Display toggle fix
@kantagara
Copy link
Contributor

kantagara commented Dec 7, 2024

We should consider shifting test networks for efficiency. B3 is fast and easy to get testnet tokens, i vote there.

Yep I've already created the ticket for that 🫠

@kantagara
Copy link
Contributor

Should work with any ERC-1155?

Do you have it on your web3auth wallet?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better :)

Copy link
Collaborator

@rob1997 rob1997 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kewl

@sneakzttv
Copy link
Contributor Author

awaiting final check with updated RPC from @juans-chainsafe

Copy link
Contributor

@juans-chainsafe juans-chainsafe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, I was using a NFT minted in a different wallet than my w3a address.
I send a ERC1155 to my w3a address and worked!
Screenshot 2024-12-09 at 9 26 05 AM

@sneakzttv sneakzttv added ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks labels Dec 9, 2024
@sneakzttv sneakzttv enabled auto-merge (squash) December 9, 2024 12:36
@sneakzttv sneakzttv disabled auto-merge December 9, 2024 12:37
@sneakzttv sneakzttv merged commit 9c72118 into dev Dec 9, 2024
7 checks passed
@sneakzttv sneakzttv deleted the sneakz/w3a-nft-tracker branch December 9, 2024 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documented Apply on Pull Request after documentation to allow merge ready-to-merge Ready to Merge PR - this'll trigger required checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants