-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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: Dynamic On-Chain Token Attributes Repository #7635
Add EIP: Dynamic On-Chain Token Attributes Repository #7635
Conversation
This proposal introduces the ability to assign attributes to NFTs in a public non-gated repository smart contract that is accessible at the same address in all of the networks. The repository smart contract is designed to be a common-good repository, meaning that it can be used by any ERC-721 or ERC-1155 compatible token.
✅ All reviewers have approved. |
Co-authored-by: Steven Pineda <steven2308@outlook.com>
The commit 8d31ce5 (as a parent of 30149de) 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.
Looks good, just need to rename the files to match the EIP number!
EIPS/eip-x.md
Outdated
@@ -0,0 +1,1075 @@ | |||
--- | |||
eip: XYZ |
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: XYZ | |
eip: 7508 |
EIPS/eip-x.md
Outdated
--- | ||
eip: XYZ | ||
title: Dynamic On-Chain Token Attributes Repository | ||
description: React to any Non-Fungible Tokens using Unicode emojis. |
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.
Maybe forgot to update the description?
EIPS/eip-x.md
Outdated
type: Standards Track | ||
category: ERC | ||
created: 2023-08-15 | ||
requires: 165 |
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.
requires: 165 | |
requires: 165, 712 |
I'd probably mention 712 here.
EIPS/eip-x.md
Outdated
|
||
### Message format for presigned emotes | ||
|
||
The message to be signed by the `setter` in order for the attribute setting to be submitted by someone else is formatted as follows: |
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.
I'd recommend mentioning where this pre-signed stuff comes from. I think it's ERC-712?
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...
This proposal introduces the ability to assign attributes to NFTs in a public non-gated repository smart contract that is accessible at the same address in all of the networks. The repository smart contract is designed to be a common-good repository, meaning that it can be used by any ERC-721 or ERC-1155 compatible token.