-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add EIP: Registry for royalties payment for NFTs #6786
Conversation
✅ All reviewers have approved. |
8d7c874
to
9468cfe
Compare
b7f71d6
to
822b976
Compare
c8e846c
to
1abcbe6
Compare
1abcbe6
to
c838d60
Compare
EIPS/eip-5666.md
Outdated
@@ -0,0 +1,105 @@ | |||
--- | |||
eip: 5666 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eip: 5666 | |
eip: 6786 |
Assigning ERC number using PR number.
c838d60
to
ac3c9e0
Compare
EIPS/eip-6786.md
Outdated
event RoyaltiesPaid(address indexed tokenAddress, uint256 indexed tokenId, uint256 amount); | ||
|
||
/// @notice sends msg.value to the creator of a NFT | ||
/// @dev Throws if there are no on-chain informations about the creator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps reverts is a better term?
/// @dev Throws if there are no on-chain informations about the creator | |
/// @dev Reverts if there are no on-chain informations about the creator |
EIPS/eip-6786.md
Outdated
|
||
All functions defined as view MAY be implemented as pure or view | ||
|
||
Function payRoyalties MAY be implemented as public or external |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function payRoyalties MAY be implemented as public or external | |
Function `payRoyalties` MAY be implemented as public or external |
EIPS/eip-6786.md
Outdated
|
||
## Rationale | ||
|
||
With a lot of places made for trading NFTs dropping down the royalty payment or having a centralised approach, we want to provide a way for anyone to pay royalties to the creators. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This paragraph belongs in the motivation section because it is justifying the EIP as a whole, and not describing a specific technical choice made within the EIP.
assets/eip-6786/LICENSE
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need to include CC0 as it's included in the root of the repository.
assets/eip-6786/README.md
Outdated
|
||
# ERC6786 Royalty Debt Registry | ||
|
||
[![License: CC0-1.0](https://img.shields.io/badge/License-CC0-yellow.svg)](https://creativecommons.org/publicdomain/zero/1.0/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove any external links.
[![License: CC0-1.0](https://img.shields.io/badge/License-CC0-yellow.svg)](https://creativecommons.org/publicdomain/zero/1.0/) |
assets/eip-6786/abi/ERC6786.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These ABI files aren't strictly necessary to understand the reference implementation, so please remove them.
assets/eip-6786/hardhat.config.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove any build system configuration files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not critical to understand the reference implementation, please remove.
The commit 2a0064f (as a parent of c8cc3fb) contains errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All Reviewers Have Approved; Performing Automatic Merge...
When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met: