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

Setting royalties as per EIP2981 / ERC2981 #301

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on May 27, 2022

  1. Setting royalties as per EIP2981 / ERC2981

    a small tweak to implement EIP2981 to set royalties.
    I understand it is up to the marketplace to honour it (e.g. OpenSea and Rarible have their own "hack" for royalties) but it seems the EIP2981 is starting to be adopted by newer marketplaces and hopefully the dinosaurs will eventually follow. 
    these changes are small and hopefully make this contract future proof
    
    as per Liarco suggestion, I've created a way to set royalties at any time, but have a default open in the construct
    
    *note: I've only used this in a couple of test contracts, so please do your own tests and submit any feedback  and/or tweaks here
    it-is-zods authored May 27, 2022
    Configuration menu
    Copy the full SHA
    a7dbbd6 View commit details
    Browse the repository at this point in the history

Commits on May 28, 2022

  1. Update YourNftToken.sol

    small typo, was missing the _
    it-is-zods authored May 28, 2022
    Configuration menu
    Copy the full SHA
    5b7929a View commit details
    Browse the repository at this point in the history