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 #11

Open
vpontis opened this issue May 3, 2022 · 1 comment
Open

Royalties #11

vpontis opened this issue May 3, 2022 · 1 comment
Milestone

Comments

@vpontis
Copy link
Collaborator

vpontis commented May 3, 2022

We will want to send royalties to multiple accounts but we don't want to store all of them on the NFT (since that will make creating the NFT more expensive for everyone)

Instead we could set up:

RoyaltyConfigAccount

  • royalty_percent
  • creators - CreatorRoyalty[]
    • type CreatorRoyalty = { creator: Address; shares: u64 }
    • The amount each creator gets is the

dispense_royalties Instruction

  • call with
    • total_fee
    • nft_account
    • royalty_config
  • it will then call invoke_signed to pay out the different accounts based on shares / royalty_percent

Questions

  1. Should this be a collection or nft level primitive?
@vpontis vpontis added this to the Post launch milestone May 3, 2022
@crypt0miester
Copy link

since almost all nfts belong to a collection it might be good to make the royalties a per collection basis.

unless it is a 1/1 which is not in a collection. then it would need to be on the nft.

perhaps make it an optional field?

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

2 participants