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

NFT Transfers Page #70

Merged
merged 6 commits into from
Aug 10, 2024
Merged

Conversation

bhavyagosai
Copy link
Contributor

  • resolves Contract page NFT transfers #39
  • added functions to ERC1155 abi
    • name()
    • symbol()
  • updated next.config.mjs to whitelist domains for image
  • added NFT transfers page:
    • for fetching nft metadata, for both ERC721 and ERC1155 contracts, tokenURI is checked if it uses https or ipfs protocol
      • getting image either from content type being application/json or image and parsing accordingly for both protocols.
      • covering an edge case where placeholder for tokenID is received in the uri itself, where we replace it with the actual tokenID.
      • showing default image / values where not able to execute tokenABI function in corresponding contract.

- convert ipfs to https function
- code format
- this was done as an nft image file is hosted online and on random domains
- added function to fetch latest NFT transfer events from ERC721 and ERC1155 contracts
- added function to fetch NFT metadata so as to get data such as name, symbol and image
- added necessary copy buttons
- added necessary tooltips
Copy link

vercel bot commented Aug 9, 2024

@bhavyagosai is attempting to deploy a commit to the Walnut Team on Vercel.

A member of the Team first needs to authorize it.

saimeunt
saimeunt previously approved these changes Aug 9, 2024
Copy link
Collaborator

@saimeunt saimeunt left a comment

Choose a reason for hiding this comment

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

@bhavyagosai Nice work, can you resolve the conflicts so we can merge please?

@saimeunt saimeunt merged commit 10941dc into walnuthq:main Aug 10, 2024
1 check failed
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.

Contract page NFT transfers
2 participants