Hi Team, I am looking at https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC721/ERC721.sol 0.8.0 version. It seems like that it doesn’t contain the function which receives the ipfs hash and metadatauri to set on a token. There was something like this in the previ…

Hi @novaknole ! This was split off into a separate contract called ERC721URIStorage, because using storage in this way is expensive and we decided to make the default mechanism significantly cheaper. You may find these threads useful: Function _setTokenURI() in ERC721 is gone with pragma ^0.8.0 Op…

Doesn’t make sense to me at all. We all agree that setTokenURI 's costs is just storing bytes32 hash into a storage for which each user pays only for his/her part in the storage. So I don’t see much costs here. If we need to append something in tokenURI, it still is view function right because i…

Hi @novaknole , I recommend trying it out on a public testnet and calculating the cost difference between storing an IPFS hash per token ID and not storing when minting.

@abcoathup @hantuzun @frangio Hey , I compared the costs for both cases : 1. without using setTokenURI and using predefined metadata vs 2. using setTokenURI. The very vague (might be wrong with 5,000 - 10,000 Gas, not that important) I found is this: without setTokenURI, and just using predefin…

Hi @novaknole , It really depends on your use case and the value of the NFTs. If the cost of minting is significant compared with the value of each token then you should be optimizing for gas costs. Question 1, It sounds like your use case requires metadata to be created at time of minting and can…

[image] frangio: https://gateway.pinata.cloud/ipfs/QmcP9hxrnC1T5ATPmq2saFeAM1ypFX9BnAswCdHB9JCjLA/ Would you suggest to deploy one ERC721 for every unique asset of one service or to deploy one contract and users mint against it for each unique asset? For example an art registry where users up…

It will be prohibitively expensive to deploy one ERC721 contract for every user, let alone for every piece of artwork. This is one of the cases where using ERC721URIStorage may make more sense. If you want to deploy a separate ERC721 contract per user for other reasons (e.g. so that each user has …

This is one of the most discussed topic around NTFs, There is so many angle to deal with this and my response is always the same. It’s all about use case. How you want your smart contract to deal with the data. The way I approach this is. Do I want a Decentralized data storage system or do I want i…

Hi, I wanted to bank off this thread as it is somewhat related. I wanted to take the approach of having the metadata predefined in a directory and uploaded under one hash to IPFS at the time of deployment; however, wouldn't this conflict with the concept of your typical NFT sale? The buyer of the f…

Yes, normally you would want the distribution/sale to be randomized so that it can't be gamed by people who know beforehand which are the rare ones. There is a relevant conversation about that here: [image] Are NFT projects doing starting index randomization and provenance wrong o…

Hi @frangio thanks for your answers! One question though, if we used IPNS, wouldn't this be centralized as it is tied to a specific private key? Wouldn't this defeat the purpose of IPFS when minting NFTs, as described in https://docs.ipfs.io/how-to/mint-nfts-with-ipfs/#a-short-introduction-to-nfts ?

Why doesn't OpenZeppelin ERC721 contain setTokenURI?

Support Contracts

Asad_Ali_Logicon November 30, 2021, 2:17pm 34

what if i want to mint Dynamically NFTs and Dynamic Meta Data

Topic		Replies	Views
Gas cost optimization in 1/n ERC721 Contracts erc721 , design	1	3050	February 2, 2021
Function _setTokenURI() in ERC721 is gone with pragma ^0.8.0 Contracts erc721	34	33924	November 15, 2021
Passing URI for ERC721 token at mint Contracts	17	5254	April 24, 2022
ERC721 - NFT - all the tokens sharing the same metadata tokenURI? Support erc721	3	7902	April 19, 2021
How has the metadata been added to the TokenID in this NFT contract? Smart Contracts erc721 , etherscan-verify	12	4285	October 1, 2021

Why doesn't OpenZeppelin ERC721 contain setTokenURI?

Related topics