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

Royalties percentage and specified wallet #12

Closed
gitrevo opened this issue Feb 20, 2022 · 4 comments
Closed

Royalties percentage and specified wallet #12

gitrevo opened this issue Feb 20, 2022 · 4 comments

Comments

@gitrevo
Copy link

gitrevo commented Feb 20, 2022

Hi again,

How can we add royalties % for secondary markets and royalties wallet address directly into this smart contract? If yes, mind to point which links/video shows this line of codes?

It would be great to assign this royalties in the contract so it will work in all secondary markets should we forget to set it up (os, looksrare, x2y2 etc.).

Thanks again for the great tutorials!

@liarco
Copy link
Member

liarco commented Feb 20, 2022

Hi @gitrevo,
thank you for your feedback. There is still no standard way to do that. OpenSea is supporting a custom way to handle it so you may try implementing it if you wish. There shouldn't be any compatibility issue. (I just suggest going for a dynamic variable as any other URI so you won't get stuck with an initial value if you need to update it)

I have to point out that it's unlikely that we will introduce non-standard features like this in the default contract unless it becomes really widely adopted.

@liarco liarco closed this as completed Feb 20, 2022
@gitrevo
Copy link
Author

gitrevo commented Feb 20, 2022

contract GitRevoNFT is ERC721 {
function contractURI() public view returns (string memory) {
return "https://gitrevo.io/contract-metadata.json";
}
}

In contract-metadata.json

{
"name": "GitRevoNFT",
"description": "This is Git Revo NFT",
"image": "https://gitrevo.io/gitrevo.png",
"external_link": "https://gitrevo.io",
"seller_fee_basis_points": 500, # Indicates a 5% seller fee.
"fee_recipient": "0xA97F337c39cccE66adfeCB2BF99C1DdC54C2D721" # Where seller fees will be paid to.
}

Will this actually works on OpenSea only? Or it will work on other marketplace (looksrare, x2y2 etc.) as well?

@liarco
Copy link
Member

liarco commented Feb 20, 2022

@gitrevo as I said:

There is still no standard way to do that. OpenSea is supporting a custom way to handle it

So that is a non-standard implementation suggested by OpenSea only. Other marketplaces might be using it as well, but to my knowledge there is no warranty that all the marketplaces are going to use it. I'm sorry.

@granthughes1999
Copy link

contract GitRevoNFT is ERC721 { function contractURI() public view returns (string memory) { return "https://gitrevo.io/contract-metadata.json"; } }

Hey @liarco or @gitrevo,

Could you explain where that https://gitrevo.io/contract-metadata.json is supposed to point to exactly?

In contract-metadata.json

{ "name": "GitRevoNFT", "description": "This is Git Revo NFT", "image": "https://gitrevo.io/gitrevo.png", "external_link": "https://gitrevo.io", "seller_fee_basis_points": 500, # Indicates a 5% seller fee. "fee_recipient": "0xA97F337c39cccE66adfeCB2BF99C1DdC54C2D721" # Where seller fees will be paid to. }

Will this actually works on OpenSea only? Or it will work on other marketplace (looksrare, x2y2 etc.) as well?

Could you also explain what to do with this .json file? Do I add contract-metadata.json as a file to the nft-erc721-collection-main ?

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

No branches or pull requests

3 participants